diff options
| author | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-04-30 12:00:39 +0530 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-04-30 12:00:39 +0530 |
| commit | bc8f194c8c29e46e8ee4034f06e46988dfff38f7 (patch) | |
| tree | fce0fda2012a3d78d9557afc4a4f5bbe5fa9ee4c /ChangeLog | |
| parent | 1cdeb2372ddecac0dfe0c132a033e9590ffa07d2 (diff) | |
| download | glibc-bc8f194c8c29e46e8ee4034f06e46988dfff38f7.tar.gz glibc-bc8f194c8c29e46e8ee4034f06e46988dfff38f7.tar.xz glibc-bc8f194c8c29e46e8ee4034f06e46988dfff38f7.zip | |
Initialize all of datahead structure in nscd (BZ #16791)
The datahead structure has an unused padding field that remains uninitialized. Valgrind prints out a warning for it on querying a netgroups entry. This is harmless, but is a potential data leak since it would result in writing out an uninitialized byte to the cache file. Besides, this happens only when there is a cache miss, so we're not adding computation to any fast path.
Diffstat (limited to 'ChangeLog')
| -rw-r--r-- | ChangeLog | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog index 942fb801e1..2b0821fd27 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2014-04-30 Siddhesh Poyarekar <siddhesh@redhat.com> + [BZ #16791] + * nscd/nscd-client.h (datahead_init_common): Initialize entire + structure. + (datahead_init_pos): Call datahead_init_common early. + (datahead_init_neg): Likewise. + * nscd/nscd-client.h (datahead_init_common, datahead_init_pos, datahead_init_neg): New functions. * nscd/aicache.c (addhstaiX): Use them. |