Do not crash if def_bd == NULL (!)

Patch by Andrei Nigmatulin.


git-svn-id: https://protobuf-c.googlecode.com/svn/trunk@266 00440858-1255-0410-a3e6-75ea37f81c3a

ChangeLog

 0.15 (NOT YET RELEASED)
       - make protobuf_c_message_init() into a function (Issue #49, daveb)
+      - Fix for freeing memory after unpacking bytes w/o a default-value.
+        (Andrei Nigmatulin)
 
 0.14:
       - build fix (missing dependency in test directory)

src/google/protobuf-c/protobuf-c.c

@@ -1751,7 +1751,9 @@ parse_required_member (ScannedMember *scanned_member,
         const ProtobufCBinaryData *def_bd;
         unsigned pref_len = scanned_member->length_prefix_len;
         def_bd = scanned_member->field->default_value;
-        if (maybe_clear && bd->data != NULL && bd->data != def_bd->data)
+        if (maybe_clear
+         && bd->data != NULL
+         && (def_bd == NULL || bd->data != def_bd->data))
           FREE (allocator, bd->data);
         DO_ALLOC (bd->data, allocator, len - pref_len, return 0);
         memcpy (bd->data, data + pref_len, len - pref_len);