OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_2-569-gc0504fc
Gerrit Code Review
gerrit@openafs.org
Mon, 24 Aug 2026 12:32:18 -0400
The following commit has been merged in the master branch:
commit 0d1656f890ce1380f8d756b7a782ec679ff3756e
Author: Junsouh Hong <junsouh@gmail.com>
Date: Sun Jul 19 23:02:04 2026 -0500
fs: Split out aclu_ParseAcl() from ParseAcl()
Introduce aclu_ParseAcl(), which does the same thing as ParseAcl() but
can return errors, returning ENOMEM errors if we fail to allocate
memory, instead of asserting.
Additionally, add goto style cleanup as part of the change of being
able to return error codes. As part of this, initialize ta->pluslist and
ta->minuslist earlier, to ensure allocated entries are freed if an error
occurs during parsing. Also add a comment block explaining
aclu_ParseAcl() since this function will be a part of the libacl API.
Change ParseAcl() to just call aclu_ParseAcl(), and assert if it
encounters an error, reflecting previous fs behavior.
Change-Id: I4f045b1cf429f1b7375e53fa8e8e7ee082a3ffa1
Co-authored-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-on: https://gerrit.openafs.org/16930
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Marcio Brito Barbosa <mbarbosa@sinenomine.net>
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
src/venus/fs.c | 81 ++++++++++++++++++++++++++++++++++++++++++++++++++--------
1 file changed, 71 insertions(+), 10 deletions(-)
--
OpenAFS Master Repository