diff options
| author | Noel Power <noel.power@suse.com> | 2019-06-18 10:47:29 +0000 |
|---|---|---|
| committer | Noel Power <npower@samba.org> | 2019-07-01 13:34:11 +0000 |
| commit | 2b7236ffe1a0454756b76f8dc302649e92cefbaa (patch) | |
| tree | 789ab1bf557adb703889ea55cb3a31e10870936d /lib/krb5_wrap | |
| parent | 8b92539997a9df8e0d217790fd82fc8f59ea50cc (diff) | |
| download | samba-2b7236ffe1a0454756b76f8dc302649e92cefbaa.tar.gz | |
lib/krb5_wrap: clang: Fix warning: Null pointer passed as an argument
Fixes:
lib/krb5_wrap/krb5_samba.c:3241:3: warning: Null pointer passed as an argument to a 'nonnull' parameter <--[clang]
memcpy(gss_cksum + 28, in_data->data, orig_length);
^
Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
Diffstat (limited to 'lib/krb5_wrap')
| -rw-r--r-- | lib/krb5_wrap/krb5_samba.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/krb5_wrap/krb5_samba.c b/lib/krb5_wrap/krb5_samba.c index eac86e6754b..37257ddfe03 100644 --- a/lib/krb5_wrap/krb5_samba.c +++ b/lib/krb5_wrap/krb5_samba.c @@ -3234,7 +3234,7 @@ static krb5_error_code ads_create_gss_checksum(krb5_data *in_data, /* [inout] */ SIVAL(gss_cksum, 20, gss_flags); - if (orig_length) { + if (orig_length && in_data->data != NULL) { SSVAL(gss_cksum, 24, 1); /* The Delegation Option identifier */ SSVAL(gss_cksum, 26, orig_length); /* Copy the kerberos KRB_CRED data */ |