>From Dominique Martinet <dominique.marti...@cea.fr>:
Dominique Martinet has uploaded a new change for review. (
https://review.gerrithub.io/346046
Change subject: display buffer overflow: Fix off by one
......................................................................
display buffer overflow: Fix off by one
Address SANitizer (ASAN) blows up on write right after the end of the
buffer.
I *think* the function should look behind for complete utf8 chars
(instead of looking at current position) because it's always called
with exactly enough room for '...' + terminating null byte.
Even when we force the overflow, it's called with b_current set after
the last character, so the function probably used to print one byte of
garbage in this case.
Change-Id: I8df14a2f51824e466600916a89bb54ab04b14635
Signed-off-by: Dominique Martinet <dominique.marti...@cea.fr>
---
M src/log/display.c
1 file changed, 8 insertions(+), 4 deletions(-)
git pull ssh://review.gerrithub.io:29419/ffilz/nfs-ganesha
refs/changes/46/346046/1
--
To view, visit https://review.gerrithub.io/346046
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8df14a2f51824e466600916a89bb54ab04b14635
Gerrit-Change-Number: 346046
Gerrit-PatchSet: 1
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Owner: Dominique Martinet <dominique.marti...@cea.fr>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
