commit ef422656a57b3f472b192691a40f48d0d72f2927
author Petteri Aimonen <jpa@git.mail.kapsi.fi> Sat Mar 07 10:25:09 2015 +0200
committer Petteri Aimonen <jpa@git.mail.kapsi.fi> Sat Mar 07 10:25:09 2015 +0200
tree d6337dc04ed820deda96279ee00e3eb3d2e7775c
parent 5c16a116ec2db9d19e42c5682cbfbf4f2b87ad87

Fix oneof submessage initialization bug.

Update issue 149
Status: FixedInGit

pb_decode.c

diff --git a/pb_decode.c b/pb_decode.c
index 9d25dc6..4d8c7cb 100644
--- a/pb_decode.c
+++ b/pb_decode.c
@@ -396,6 +396,10 @@
 
         case PB_HTYPE_ONEOF:
             *(pb_size_t*)iter->pSize = iter->pos->tag;
+            if (PB_LTYPE(type) == PB_LTYPE_SUBMESSAGE)
+            {
+                pb_message_set_to_defaults((const pb_field_t*)iter->pos->ptr, iter->pData);
+            }
             return func(stream, iter->pos, iter->pData);
 
         default:

tests/oneof/decode_oneof.c

diff --git a/tests/oneof/decode_oneof.c b/tests/oneof/decode_oneof.c
index 83b4702..37075cd 100644
--- a/tests/oneof/decode_oneof.c
+++ b/tests/oneof/decode_oneof.c
@@ -2,6 +2,7 @@
 
 #include <stdio.h>
 #include <stdlib.h>
+#include <string.h>
 #include <pb_decode.h>
 #include "oneof.pb.h"
 #include "test_helpers.h"
@@ -10,9 +11,12 @@
 /* Test the 'OneOfMessage' */
 int test_oneof_1(pb_istream_t *stream, int option)
 {
-    OneOfMessage msg = OneOfMessage_init_zero;
+    OneOfMessage msg;
     int status = 0;
 
+    /* To better catch initialization errors */
+    memset(&msg, 0xAA, sizeof(msg));
+
     if (!pb_decode(stream, OneOfMessage_fields, &msg))
     {
         printf("Decoding failed: %s\n", PB_GET_ERROR(stream));
@@ -124,4 +128,4 @@
     }
 
     return 0;
-}
\ No newline at end of file
+}