On Sun, Jun 23, 2013 at 04:23:14PM -0700, Tyler Hicks wrote: > From: John Johansen <[email protected]> > > Return the total size of the security context on success > as documented. > > Signed-off-by: John Johansen <[email protected]>
The implementation of this idea is fine. But the *size parameter is already being updated with the length, which seems sufficient to me. Thanks > --- > libraries/libapparmor/src/kernel_interface.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libraries/libapparmor/src/kernel_interface.c > b/libraries/libapparmor/src/kernel_interface.c > index 1c3b4df..ee7908a 100644 > --- a/libraries/libapparmor/src/kernel_interface.c > +++ b/libraries/libapparmor/src/kernel_interface.c > @@ -583,6 +583,8 @@ int aa_getpeercon_raw(int fd, char *buffer, int *size) > optlen++; > } > } > + > + rc = optlen; > out: > *size = optlen; > return rc; > -- > 1.8.1.2 > > > -- > AppArmor mailing list > [email protected] > Modify settings or unsubscribe at: > https://lists.ubuntu.com/mailman/listinfo/apparmor >
signature.asc
Description: Digital signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
