diff libfdproto/messages.c @ 752:9e9840ccf059

Attempt to handle more gracefully (invalid) AVPs that contain an empty octetstring
author Sebastien Decugis <sdecugis@nict.go.jp>
date Sat, 24 Sep 2011 09:45:14 +0200
parents 4a9f08d6b6ba
children 71833fa5e35f
line wrap: on
line diff
--- a/libfdproto/messages.c	Thu Sep 22 21:10:00 2011 +0200
+++ b/libfdproto/messages.c	Sat Sep 24 09:45:14 2011 +0200
@@ -1770,12 +1770,8 @@
 					return EBADMSG;
 				} );
 			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;
 		
 		case AVP_TYPE_INTEGER32:
"Welcome to our mercurial repository"