[18872] in Kerberos_V5_Development

home help back first fref pref prev next nref lref last post

krb5_free_data

daemon@ATHENA.MIT.EDU (Markus Moeller)
Sat Jun 21 12:56:58 2014

To: krbdev@mit.edu
From: "Markus Moeller" <huaraz@moeller.plus.com>
Date: Sat, 21 Jun 2014 17:56:23 +0100
Message-ID: <lo4dft$ue3$1@ger.gmane.org>
Mime-Version: 1.0
X-Complaints-To: usenet@ger.gmane.org
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: krbdev-bounces@mit.edu

Hi,

   I wonder if it wouldn't be  better to check if val->data is NULL before 
freeing ?  To me the function makes otherwise no sense as I have to do the 
same checks myself before  I can call the function safely.


void KRB5_CALLCONV
krb5_free_data(krb5_context context, krb5_data *val)
{
    if (val == NULL)
        return;
    free(val->data);
    free(val);
}


Thank you
Markus 


_______________________________________________
krbdev mailing list             krbdev@mit.edu
https://mailman.mit.edu/mailman/listinfo/krbdev

home help back first fref pref prev next nref lref last post