The function in question does not just verify the header but also does unpacking, so give the function a more descriptive name.
Signed-off-by: Ryan Lee <ryan....@canonical.com> --- security/apparmor/policy_unpack.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/security/apparmor/policy_unpack.c b/security/apparmor/policy_unpack.c index 6cf6e372d815..22af940a5f58 100644 --- a/security/apparmor/policy_unpack.c +++ b/security/apparmor/policy_unpack.c @@ -1200,14 +1200,14 @@ static struct aa_profile *unpack_profile(struct aa_ext *e, char **ns_name) } /** - * verify_header - unpack serialized stream header + * unpack_and_verify_header - unpack serialized stream header * @e: serialized data read head (NOT NULL) * @required: whether the header is required or optional * @ns: Returns - namespace if one is specified else NULL (NOT NULL) * * Returns: error or 0 if header is good */ -static int verify_header(struct aa_ext *e, int required, const char **ns) +static int unpack_and_verify_header(struct aa_ext *e, int required, const char **ns) { int error = -EPROTONOSUPPORT; const char *name = NULL; @@ -1504,7 +1504,7 @@ int aa_unpack(struct aa_loaddata *udata, struct list_head *lh, *ns = NULL; while (e.pos < e.end) { void *start; - error = verify_header(&e, e.pos == e.start, ns); + error = unpack_and_verify_header(&e, e.pos == e.start, ns); if (error) goto fail; -- 2.43.0