Changeset 752:9e9840ccf059 in freeDiameter for libfdproto/messages.c

Timestamp:

Sep 24, 2011, 4:45:14 PM (13 years ago)

Author:

Sebastien Decugis <sdecugis@nict.go.jp>

Branch:

default

Phase:

public

Message:

Attempt to handle more gracefully (invalid) AVPs that contain an empty octetstring

File:

• libfdproto/messages.c (modified) (1 diff)

libfdproto/messages.c

@@ -1771 +1771 @@
                                 } );
                         avp->avp_storage.os.len = avp->avp_public.avp_len - GETAVPHDRSZ( avp->avp_public.avp_flags );
-                        if (avp->avp_storage.os.len) {
-                                CHECK_MALLOC(  avp->avp_storage.os.data = os0dup(avp->avp_source, avp->avp_storage.os.len)  );
-                                avp->avp_mustfreeos = 1;
-                        } else {
-                                avp->avp_storage.os.data = NULL;
-                        }
+                        CHECK_MALLOC(  avp->avp_storage.os.data = os0dup(avp->avp_source, avp->avp_storage.os.len)  );
+                        avp->avp_mustfreeos = 1;
                         break;