Commit c66080ad authored by J. Bruce Fields's avatar J. Bruce Fields

svcrpc: fix gss-proxy NULL dereference in some error cases

We depend on the xdr decoder to set this pointer, but if we error out
before we decode this piece it could be left NULL.

I think this is probably tough to hit without a buggy gss-proxy.
Reported-by: default avatarAndi Kleen <andi@firstfloor.org>
Cc: Simo Sorce <simo@redhat.com>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent 97e47fa1
...@@ -298,6 +298,7 @@ int gssp_accept_sec_context_upcall(struct net *net, ...@@ -298,6 +298,7 @@ int gssp_accept_sec_context_upcall(struct net *net,
if (res.context_handle) { if (res.context_handle) {
data->out_handle = rctxh.exported_context_token; data->out_handle = rctxh.exported_context_token;
data->mech_oid.len = rctxh.mech.len; data->mech_oid.len = rctxh.mech.len;
if (rctxh.mech.data)
memcpy(data->mech_oid.data, rctxh.mech.data, memcpy(data->mech_oid.data, rctxh.mech.data,
data->mech_oid.len); data->mech_oid.len);
client_name = rctxh.src_name.display_name; client_name = rctxh.src_name.display_name;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment