diff options
| author | Steve Dickson <steved@redhat.com> | 2011-07-20 09:47:49 -0400 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2011-07-20 11:21:32 -0400 |
| commit | e6bdc5761e5340b0df03c2b5c344b04feabaad9e (patch) | |
| tree | e93dd3c3d83e3f7b4ffaa76ed177e0ce2a05874a /src/svc_dg.c | |
| parent | ebd3e644a79fa92b3b780d87d8028fcf64364abd (diff) | |
| download | ti-rpc-e6bdc5761e5340b0df03c2b5c344b04feabaad9e.tar.gz | |
Segfault in SVCAUTH_WRAP calllibtirpc-0-2-3-rc2
The xprt->xp_auth pointer need to be checked before
used in the SVCAUTH_WRAP call since it can be NULL
during error conditions.
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'src/svc_dg.c')
| -rw-r--r-- | src/svc_dg.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/svc_dg.c b/src/svc_dg.c index 5ef9df2..081db61 100644 --- a/src/svc_dg.c +++ b/src/svc_dg.c @@ -254,8 +254,8 @@ svc_dg_reply(xprt, msg) XDR_SETPOS(xdrs, 0); msg->rm_xid = su->su_xid; if (xdr_replymsg(xdrs, msg) && - (!has_args || - (SVCAUTH_WRAP(xprt->xp_auth, xdrs, xdr_results, xdr_location)))) { + (!has_args || (xprt->xp_auth && + SVCAUTH_WRAP(xprt->xp_auth, xdrs, xdr_results, xdr_location)))) { struct msghdr *msg = &su->su_msghdr; struct iovec iov; |