Variation on a theme, with same result - double free of same object.

Coverity #950
Signed-off-by: Dave Jones <[EMAIL PROTECTED]>

--- linux-2.6/net/sunrpc/auth_gss/gss_krb5_seal.c~      2006-03-06 
04:23:05.000000000 -0500
+++ linux-2.6/net/sunrpc/auth_gss/gss_krb5_seal.c       2006-03-06 
04:23:59.000000000 -0500
@@ -133,12 +133,12 @@ gss_get_mic_kerberos(struct gss_ctx *gss
                BUG();
        }
 
-       kfree(md5cksum.data);
-
        if ((krb5_make_seq_num(ctx->seq, ctx->initiate ? 0 : 0xff,
                               ctx->seq_send, krb5_hdr + 16, krb5_hdr + 8)))
                goto out_err;
 
+       kfree(md5cksum.data);
+
        ctx->seq_send++;
 
        return ((ctx->endtime < now) ? GSS_S_CONTEXT_EXPIRED : GSS_S_COMPLETE);


-- 
http://www.codemonkey.org.uk
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to