diff options
author | Josh Haberman <jhaberman@gmail.com> | 2019-01-28 12:07:06 -0800 |
---|---|---|
committer | Josh Haberman <jhaberman@gmail.com> | 2019-01-28 12:07:06 -0800 |
commit | 22ad7121c2792e3c473776c22f8e7a63a0116d33 (patch) | |
tree | e6920484d20e7bbf9712d0724b01a058a33d81ad /upb/def.c | |
parent | 8e737d4f9acd3bb7620fdb237f5b943b41b75e9e (diff) |
Fixes for google3.
Diffstat (limited to 'upb/def.c')
-rw-r--r-- | upb/def.c | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -4,7 +4,6 @@ #include <ctype.h> #include <stdlib.h> #include <string.h> -#include "upb/structdefs.int.h" #include "upb/handlers.h" typedef struct { @@ -663,6 +662,8 @@ static void freefield(upb_refcounted *r) { upb_fielddef_uninit_default(f); if (f->subdef_is_symbolic) upb_gfree(f->sub.name); + if (f->msg_is_symbolic) + upb_gfree(f->msg.name); upb_def_uninit(upb_fielddef_upcast_mutable(f)); upb_gfree(f); } @@ -1020,7 +1021,7 @@ bool upb_fielddef_setnumber(upb_fielddef *f, uint32_t number, upb_status *s) { s, "cannot change field number after adding to a message"); return false; } - if (number == 0 || number > UPB_MAX_FIELDNUMBER) { + if (number == 0) { upb_status_seterrf(s, "invalid field number (%u)", number); return false; } |