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