The following commit has been merged in the openafs-stable-1_6_x branch:
commit a78e3a7e673d5d0428e769a8a9f0ea1ccb683d40
Author: Simon Wilkinson <[email protected]>
Date: Sat Mar 2 11:02:09 2013 +0000
libadmin: Don't free garbage pointer
If we jump to the error handler early on in pts_GroupOwnerChange,
idlist may not have been used, and so we will end up trying to
free stack garbage.
Initialise the structure to 0 at the start of the function, so it
is always safe to enter the error handler.
Caught by coverity (#985962)
Reviewed-on: http://gerrit.openafs.org/9398
Tested-by: BuildBot <[email protected]>
Reviewed-by: Derrick Brashear <[email protected]>
Reviewed-by: Jeffrey Altman <[email protected]>
(cherry picked from commit 8260d86dda766a21e9f457994e7a3b259ba3a31b)
Change-Id: I79f7fe3cba11a2904c644e6747511e3630d74a79
Reviewed-on: http://gerrit.openafs.org/11047
Reviewed-by: Andrew Deason <[email protected]>
Tested-by: BuildBot <[email protected]>
Reviewed-by: Chas Williams - CONTRACTOR <[email protected]>
Reviewed-by: Stephan Wiesand <[email protected]>
src/libadmin/pts/afs_ptsAdmin.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
--
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs
