OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_0-90-g444a971

Gerrit Code Review gerrit@openafs.org
Wed, 13 Jan 2021 00:38:09 -0500


The following commit has been merged in the master branch:
commit 444a971edc47c34efbefed6e332ee6e843ae072b
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Sat Jan 9 12:50:03 2021 -0600

    afs: Remove SRXAFSCB_GetDE
    
    The GetDE RPC has been commented out from afscbint.xg effectively
    since it was introduced, but we still define the SRXAFSCB_GetDE server
    stub for it.
    
    This is useless, but also potentially dangerous, since the stub
    routine just returns success, without populating the output arguments.
    One of the output arguments is a string, and so if this RPC is
    actually run, the rxgen-generated server code will try to xdr_string()
    that string. Since we never set it to anything, this will result in
    xdr_string trying to dereference a NULL pointer.
    
    None of this actually happens currently, since the GetDE RPC is
    commented out. But to avoid the above situation if it's ever
    uncommented, remove the useless SRXAFSCB_GetDE function.
    
    Change-Id: I6ef478ee69a8de1ac14baa86aa82489181d67452
    Reviewed-on: https://gerrit.openafs.org/14488
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/afs_callback.c |   11 -----------
 1 files changed, 0 insertions(+), 11 deletions(-)

-- 
OpenAFS Master Repository