Whamcloud - gitweb
LU-4423 gss: fix checkpatch warning 18/10118/3
authorOleg Drokin <oleg.drokin@intel.com>
Sat, 26 Apr 2014 20:40:06 +0000 (16:40 -0400)
committerOleg Drokin <oleg.drokin@intel.com>
Mon, 5 May 2014 17:50:41 +0000 (17:50 +0000)
sizeof *null_context_new is a bad coding style.

Change-Id: Ie71803940347bd4b92d502a547d046a065bb4387
Signed-off-by: Oleg Drokin <oleg.drokin@intel.com>
Reviewed-on: http://review.whamcloud.com/10118
Tested-by: Jenkins
Reviewed-by: James Simmons <uja.ornl@gmail.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
lustre/ptlrpc/gss/gss_null_mech.c
lustre/ptlrpc/gss/gss_sk_mech.c

index 5565773..cdb120d 100644 (file)
@@ -77,7 +77,7 @@ __u32 gss_copy_reverse_context_null(struct gss_ctx *gss_context_old,
                return GSS_S_FAILURE;
 
        null_context_old = gss_context_old->internal_ctx_id;
-       memcpy(null_context_new, null_context_old, sizeof *null_context_new);
+       memcpy(null_context_new, null_context_old, sizeof(*null_context_new));
        gss_context_new->internal_ctx_id = null_context_new;
        CDEBUG(D_SEC, "succesfully copied reverse null context\n");
 
index ae9a11f..590a839 100644 (file)
@@ -77,7 +77,7 @@ __u32 gss_copy_reverse_context_sk(struct gss_ctx *gss_context_old,
                return GSS_S_FAILURE;
 
        sk_context_old = gss_context_old->internal_ctx_id;
-       memcpy(sk_context_new, sk_context_old, sizeof *sk_context_new);
+       memcpy(sk_context_new, sk_context_old, sizeof(*sk_context_new));
        gss_context_new->internal_ctx_id = sk_context_new;
        CDEBUG(D_SEC, "succesfully copied reverse sk context\n");