Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix leaked token text #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Mar 5, 2023

  1. fix leaked token text

    When running with -fsanitize=leak enabled nasm prints this error:
    
    Direct leak of 3795 byte(s) in 5 object(s) allocated from:
        #0 0x7f1ff9313867 in __interceptor_malloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:145
        netwide-assembler#1 0x56000dc80bc4 in nasm_malloc nasmlib/alloc.c:55
        netwide-assembler#2 0x56000dcd526b in new_Token asm/preproc.c:1879
        netwide-assembler#3 0x56000dcd4667 in tokenize asm/preproc.c:1748
        netwide-assembler#4 0x56000dd0809f in pp_tokline asm/preproc.c:7718
        netwide-assembler#5 0x56000dd09715 in pp_getline asm/preproc.c:7834
        netwide-assembler#6 0x56000dc7c63c in assemble_file asm/nasm.c:1722
        netwide-assembler#7 0x56000dc754e4 in main asm/nasm.c:719
        netwide-assembler#8 0x7f1ff8a2bd8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58
        netwide-assembler#9 0x7f1ff8a2be3f in __libc_start_main_impl ../csu/libc-start.c:392
        netwide-assembler#10 0x56000dc70e04 in _start (/home/ivan/d/nasm/nasm+0x2e2e04)
    
    This error was reproducible on testnos3.asm test.
    
    Signed-off-by: Ivan Sorokin <[email protected]>
    sorokin committed Mar 5, 2023
    Configuration menu
    Copy the full SHA
    a8ef28f View commit details
    Browse the repository at this point in the history