-
Notifications
You must be signed in to change notification settings - Fork 657
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
Little-Big Endian #2
Comments
There is an issue compiling for ppc64el as of 62bbb67. batesste@ubuntu-ppc:~/nvme-cli$ make nvme |
I think all the le*toh's are in place now. Will close this. |
Fixes: ==15224==ERROR: AddressSanitizer: stack-buffer-overflow on address 0x7fffffffcf55 at pc 0x7ffff6ecb9f5 bp 0x7fffffffcd80 sp 0x7fffffffc510 WRITE of size 22 at 0x7fffffffcf55 thread T0 #0 0x7ffff6ecb9f4 in __interceptor_vsprintf (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x619f4) #1 0x7ffff6ecbcc9 in __interceptor_sprintf (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x61cc9) linux-nvme#2 0x420357 in json_nvme_id_ctrl /home/sbauer/nvme_code/nvme-cli/nvme-print.c:1266 linux-nvme#3 0x4072f1 in __id_ctrl /home/sbauer/nvme_code/nvme-cli/nvme.c:935 linux-nvme#4 0x40742f in id_ctrl /home/sbauer/nvme_code/nvme-cli/nvme.c:950 linux-nvme#5 0x42fa63 in handle_plugin /home/sbauer/nvme_code/nvme-cli/plugin.c:140 linux-nvme#6 0x410c46 in main /home/sbauer/nvme_code/nvme-cli/nvme.c:2729 linux-nvme#7 0x7ffff6ac182f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f) linux-nvme#8 0x402268 in _start (/home/sbauer/nvme_code/nvme-cli/nvme+0x402268) Address 0x7fffffffcf55 is located in stack of thread T0 at offset 117 in frame #0 0x42011f in json_nvme_id_ctrl /home/sbauer/nvme_code/nvme-cli/nvme-print.c:1254 This frame has 3 object(s): [32, 41) 'fr' [96, 117) 'sn' <== Memory access at offset 117 overflows this variable [160, 201) 'mn' Signed-off-by: Scott Bauer <scott.bauer@intel.com>
I know it's crazy, but I hear some people don't use x86. I don't want those folks to feel dejected, so I'll post the issue so they know the little people are looking out for the big people (pun intended).
The text was updated successfully, but these errors were encountered: