Skip to content

Conversation

@geofft
Copy link
Collaborator

@geofft geofft commented Nov 6, 2025

No description provided.

@geofft
Copy link
Collaborator Author

geofft commented Nov 6, 2025

This is very badly implemented, I just want to see how far it gets with this much effort.

@jjhelmus
Copy link
Contributor

jjhelmus commented Nov 6, 2025

Would a debug+asan configuration make more sense then pgo+lto+asan? Is there many cases where you want to find memory issues in an optimized build?

@jjhelmus
Copy link
Contributor

jjhelmus commented Nov 6, 2025

Interesting. ncurses looks to have a memory leak that is triggered when running tic

2025-11-06T20:20:19.4276994Z ncurses> 1833 entries written to /build/out/usr/share/terminfo
2025-11-06T20:20:19.5434638Z ncurses> =================================================================
2025-11-06T20:20:19.5438853Z ncurses> ==6084==ERROR: LeakSanitizer: detected memory leaks
2025-11-06T20:20:19.5439440Z ncurses> 
2025-11-06T20:20:19.5439788Z ncurses> Direct leak of 1066464 byte(s) in 322 object(s) allocated from:
2025-11-06T20:20:19.5442104Z ncurses>     #0 0x5616aae2b5f4 in malloc /build/llvm/compiler-rt/lib/asan/asan_malloc_linux.cpp:67:3
2025-11-06T20:20:19.5443653Z ncurses>     #1 0x5616aaeba30d in _nc_init_termtype (/build/ncurses-6.5/progs/tic+0x19130d)
2025-11-06T20:20:19.5447281Z ncurses>     #2 0x5616aaeee34c in _nc_init_entry (/build/ncurses-6.5/progs/tic+0x1c534c)
2025-11-06T20:20:19.5452424Z ncurses>     #3 0x5616aaed2787 in _nc_parse_entry (/build/ncurses-6.5/progs/tic+0x1a9787)
2025-11-06T20:20:19.5453104Z ncurses>     #4 0x5616aaecc56c in _nc_read_entry_source (/build/ncurses-6.5/progs/tic+0x1a356c)
2025-11-06T20:20:19.5453624Z ncurses>     #5 0x5616aae6f6f2 in main (/build/ncurses-6.5/progs/tic+0x1466f2)
2025-11-06T20:20:19.5454225Z ncurses>     #6 0x7f13fab43b44 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x21b44) (BuildId: 48c48bc6abb794461b8a558dd76b29876a0551f0)
2025-11-06T20:20:19.5454723Z ncurses> 

@geofft
Copy link
Collaborator Author

geofft commented Nov 7, 2025

Looks like this was previously reported in https://lists.gnu.org/archive/html/bug-ncurses/2024-11/msg00013.html and not really acted upon. I'm going to turn off leak detection in the build for now (and might turn off all sanitizers if this doesn't get far enough).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants