author | Ezra Peisach <epeisach@med-xtal.bu.edu> | |
Tue, 19 Jun 2007 06:05:47 +0000 (08:05 +0200) | ||
committer | Florian Forster <octo@huhu.verplant.org> | |
Tue, 19 Jun 2007 06:05:47 +0000 (08:05 +0200) | ||
commit | bc5591c27f011b587d0816e737f5bc57621d18c0 | |
tree | 9978425a1ce256935ff94dda50e7b7fe7b890c7c | tree | snapshot |
parent | 92120c1d5536bae6bb50acfbe678c406283f5b56 | commit | diff |
csv, rrdtool plugin: Initialize a buffer correctly.
While reading the source code, I found the following:
In csv.c (line 44) and rrdtool.c (line 363) (value_list_to_string) the
following syntax exists:
memset (buffer, '\0', sizeof (buffer_len));
The sizeof should be removed - assuming that the desire is to zero the buffer
and not the first four (sizeof(int)) bytes.
Based on the code paths - I do not see an information leak - or any security
implications...
While reading the source code, I found the following:
In csv.c (line 44) and rrdtool.c (line 363) (value_list_to_string) the
following syntax exists:
memset (buffer, '\0', sizeof (buffer_len));
The sizeof should be removed - assuming that the desire is to zero the buffer
and not the first four (sizeof(int)) bytes.
Based on the code paths - I do not see an information leak - or any security
implications...
ChangeLog | diff | blob | history | |
src/csv.c | diff | blob | history | |
src/rrdtool.c | diff | blob | history |