commit e5049225e4fe5fd2cd12af932b75e9076013cee9
author Mike Salvatore <mike.salvatore@canonical.com> Tue Apr 09 00:44:18 2019 -0400
committer Mike Salvatore <mike.salvatore@canonical.com> Fri Apr 12 12:06:34 2019 -0400
tree 405187ed4e45319b7dae0e740f34e58fb597d8db
parent 7af7f6ebfd5010737509f2d586bfd39d6c8a2b91

apparmor: reset pos on failure to unpack for various functions

Each function that manipulates the aa_ext struct should reset it's "pos"
member on failure. This ensures that, on failure, no changes are made to
the state of the aa_ext struct.

A bug was found in unpack_u32(), unpack_u64(), unpack_array(), and
unpack_blob() where if the call to inbounds() fails, the pos member of
the aa_ext struct is not reset. All of the aforementioned functions have
been updated so that on failure the pos member of the aa_ext struct is
reset.

Signed-off-by: Mike Salvatore <mike.salvatore@canonical.com>
Acked-by: Eric Chiang <ericchiang@google.com>