diff --git a/storage/connect/tabjson.cpp b/storage/connect/tabjson.cpp index ce731971e33..78bb47c45a3 100644 --- a/storage/connect/tabjson.cpp +++ b/storage/connect/tabjson.cpp @@ -507,7 +507,15 @@ bool JSONCOL::ParseJpath(PGLOBAL g) if (Parsed) return false; // Already done - else if (InitValue(g)) + + if (InitValue(g)) + return true; + else + MulVal = Value; + + Value = NULL; + + if (InitValue(g)) return true; else if (!Jpath) Jpath = Name; @@ -540,7 +548,6 @@ bool JSONCOL::ParseJpath(PGLOBAL g) } // endfor i, p - MulVal = AllocateValue(g, Value); Parsed = true; return false; } // end of ParseJpath diff --git a/storage/connect/value.cpp b/storage/connect/value.cpp index e80c31a2baa..32a4959df06 100644 --- a/storage/connect/value.cpp +++ b/storage/connect/value.cpp @@ -436,6 +436,9 @@ PVAL AllocateValue(PGLOBAL g, PVAL valp, int newtype, int uns) bool un = (uns < 0) ? false : (uns > 0) ? true : valp->IsUnsigned(); PVAL vp; + if (!valp) + return NULL; + if (newtype == TYPE_VOID) // Means allocate a value of the same type newtype = valp->GetType(); @@ -443,13 +446,8 @@ PVAL AllocateValue(PGLOBAL g, PVAL valp, int newtype, int uns) case TYPE_STRING: p = (PSZ)PlugSubAlloc(g, NULL, 1 + valp->GetValLen()); - if ((sp = valp->GetCharString(p)) != p) { - if (sp) - strcpy (p, sp); - else - *p = 0; - - } // endif sp + if ((sp = valp->GetCharString(p)) != p) + strcpy(p, sp); vp = new(g) TYPVAL(g, p, valp->GetValLen(), valp->GetValPrec()); break;