[Bugs] [Bug 1193929] GlusterFS can be improved
bugzilla at redhat.com
bugzilla at redhat.com
Fri Aug 31 06:16:48 UTC 2018
https://bugzilla.redhat.com/show_bug.cgi?id=1193929
--- Comment #313 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/20926 committed in master by "Amar Tumballi"
<amarts at redhat.com> with a commit message- changetimerecoder xlator:
strncpy()->sprintf(), reduce strlen()'s
xlators/features/changetimerecorder/src/changetimerecorder.c
xlators/features/changetimerecorder/src/ctr-helper.h
strncpy may not be very efficient for short strings copied into
a large buffer: If the length of src is less than n,
strncpy() writes additional null bytes to dest to ensure
that a total of n bytes are written.
Instead, use snprintf(). Check for output truncation
where applicable.
Also:
- save the result of strlen() and re-use it when possible.
- move from strlen to SLEN (sizeof() ) for const strings.
Compile-tested only!
Change-Id: I2dd5629183222da8c9251af43b8b29aacf12a20a
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul at redhat.com>
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
More information about the Bugs
mailing list