* [PATCH] qobject: Fix maybe uninitialized in qdict_array_split @ 2021-05-18 13:06 Janosch Frank 2021-05-31 9:21 ` Janosch Frank 0 siblings, 1 reply; 4+ messages in thread From: Janosch Frank @ 2021-05-18 13:06 UTC (permalink / raw) To: qemu-devel; +Cc: pbonzini, cohuck, borntraeger Lets make the compiler happy. Signed-off-by: Janosch Frank <frankja@linux.ibm.com> --- qobject/block-qdict.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qobject/block-qdict.c b/qobject/block-qdict.c index 1487cc5dd8..b26524429c 100644 --- a/qobject/block-qdict.c +++ b/qobject/block-qdict.c @@ -224,7 +224,7 @@ void qdict_array_split(QDict *src, QList **dst) for (i = 0; i < UINT_MAX; i++) { QObject *subqobj; bool is_subqdict; - QDict *subqdict; + QDict *subqdict = NULL; char indexstr[32], prefix[32]; size_t snprintf_ret; -- 2.30.2 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] qobject: Fix maybe uninitialized in qdict_array_split 2021-05-18 13:06 [PATCH] qobject: Fix maybe uninitialized in qdict_array_split Janosch Frank @ 2021-05-31 9:21 ` Janosch Frank 2021-05-31 9:44 ` Philippe Mathieu-Daudé 0 siblings, 1 reply; 4+ messages in thread From: Janosch Frank @ 2021-05-31 9:21 UTC (permalink / raw) To: qemu-devel; +Cc: pbonzini, cohuck, borntraeger On 5/18/21 3:06 PM, Janosch Frank wrote: > Lets make the compiler happy. > > Signed-off-by: Janosch Frank <frankja@linux.ibm.com> Ping My build is still breaking on Ubuntu because of this. > --- > qobject/block-qdict.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/qobject/block-qdict.c b/qobject/block-qdict.c > index 1487cc5dd8..b26524429c 100644 > --- a/qobject/block-qdict.c > +++ b/qobject/block-qdict.c > @@ -224,7 +224,7 @@ void qdict_array_split(QDict *src, QList **dst) > for (i = 0; i < UINT_MAX; i++) { > QObject *subqobj; > bool is_subqdict; > - QDict *subqdict; > + QDict *subqdict = NULL; > char indexstr[32], prefix[32]; > size_t snprintf_ret; > > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] qobject: Fix maybe uninitialized in qdict_array_split 2021-05-31 9:21 ` Janosch Frank @ 2021-05-31 9:44 ` Philippe Mathieu-Daudé 2021-05-31 10:13 ` Janosch Frank 0 siblings, 1 reply; 4+ messages in thread From: Philippe Mathieu-Daudé @ 2021-05-31 9:44 UTC (permalink / raw) To: Janosch Frank, qemu-devel; +Cc: QEMU Trivial, pbonzini, cohuck, borntraeger On 5/31/21 11:21 AM, Janosch Frank wrote: > On 5/18/21 3:06 PM, Janosch Frank wrote: >> Lets make the compiler happy. >> >> Signed-off-by: Janosch Frank <frankja@linux.ibm.com> > > Ping > My build is still breaking on Ubuntu because of this. > >> --- >> qobject/block-qdict.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/qobject/block-qdict.c b/qobject/block-qdict.c >> index 1487cc5dd8..b26524429c 100644 >> --- a/qobject/block-qdict.c >> +++ b/qobject/block-qdict.c >> @@ -224,7 +224,7 @@ void qdict_array_split(QDict *src, QList **dst) >> for (i = 0; i < UINT_MAX; i++) { >> QObject *subqobj; >> bool is_subqdict; >> - QDict *subqdict; >> + QDict *subqdict = NULL; >> char indexstr[32], prefix[32]; >> size_t snprintf_ret; Slightly clearer: -- >8 -- diff --git a/qobject/block-qdict.c b/qobject/block-qdict.c index 1487cc5dd8b..8d0f00bc3ce 100644 --- a/qobject/block-qdict.c +++ b/qobject/block-qdict.c @@ -224,7 +224,6 @@ void qdict_array_split(QDict *src, QList **dst) for (i = 0; i < UINT_MAX; i++) { QObject *subqobj; bool is_subqdict; - QDict *subqdict; char indexstr[32], prefix[32]; size_t snprintf_ret; @@ -249,14 +248,16 @@ void qdict_array_split(QDict *src, QList **dst) } if (is_subqdict) { + QDict *subqdict = NULL; + qdict_extract_subqdict(src, &subqdict, prefix); assert(qdict_size(subqdict) > 0); + qlist_append_obj(*dst, QOBJECT(subqdict)); } else { qobject_ref(subqobj); qdict_del(src, indexstr); + qlist_append_obj(*dst, subqobj); } - - qlist_append_obj(*dst, subqobj ?: QOBJECT(subqdict)); } } --- Anyhow, Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Cc'ing qemu-trivial@ ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] qobject: Fix maybe uninitialized in qdict_array_split 2021-05-31 9:44 ` Philippe Mathieu-Daudé @ 2021-05-31 10:13 ` Janosch Frank 0 siblings, 0 replies; 4+ messages in thread From: Janosch Frank @ 2021-05-31 10:13 UTC (permalink / raw) To: Philippe Mathieu-Daudé, qemu-devel Cc: QEMU Trivial, pbonzini, cohuck, borntraeger On 5/31/21 11:44 AM, Philippe Mathieu-Daudé wrote: > On 5/31/21 11:21 AM, Janosch Frank wrote: >> On 5/18/21 3:06 PM, Janosch Frank wrote: >>> Lets make the compiler happy. >>> >>> Signed-off-by: Janosch Frank <frankja@linux.ibm.com> >> >> Ping >> My build is still breaking on Ubuntu because of this. >> >>> --- >>> qobject/block-qdict.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/qobject/block-qdict.c b/qobject/block-qdict.c >>> index 1487cc5dd8..b26524429c 100644 >>> --- a/qobject/block-qdict.c >>> +++ b/qobject/block-qdict.c >>> @@ -224,7 +224,7 @@ void qdict_array_split(QDict *src, QList **dst) >>> for (i = 0; i < UINT_MAX; i++) { >>> QObject *subqobj; >>> bool is_subqdict; >>> - QDict *subqdict; >>> + QDict *subqdict = NULL; >>> char indexstr[32], prefix[32]; >>> size_t snprintf_ret; > > Slightly clearer:> > -- >8 -- > diff --git a/qobject/block-qdict.c b/qobject/block-qdict.c > index 1487cc5dd8b..8d0f00bc3ce 100644 > --- a/qobject/block-qdict.c > +++ b/qobject/block-qdict.c > @@ -224,7 +224,6 @@ void qdict_array_split(QDict *src, QList **dst) > for (i = 0; i < UINT_MAX; i++) { > QObject *subqobj; > bool is_subqdict; > - QDict *subqdict; > char indexstr[32], prefix[32]; > size_t snprintf_ret; > > @@ -249,14 +248,16 @@ void qdict_array_split(QDict *src, QList **dst) > } > > if (is_subqdict) { > + QDict *subqdict = NULL; > + > qdict_extract_subqdict(src, &subqdict, prefix); > assert(qdict_size(subqdict) > 0); > + qlist_append_obj(*dst, QOBJECT(subqdict)); > } else { > qobject_ref(subqobj); > qdict_del(src, indexstr); > + qlist_append_obj(*dst, subqobj); > } > - > - qlist_append_obj(*dst, subqobj ?: QOBJECT(subqdict)); > } > } > --- If you want post that snippet yourself, go ahead. After all I only fixed a symptom without a closer look into the code. > > Anyhow, > Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> > > Cc'ing qemu-trivial@ > If not I'll take it, test on s390 and send a v2 with qemu-trivial and you in CC. Thanks for having a look! ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-05-31 10:14 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-05-18 13:06 [PATCH] qobject: Fix maybe uninitialized in qdict_array_split Janosch Frank 2021-05-31 9:21 ` Janosch Frank 2021-05-31 9:44 ` Philippe Mathieu-Daudé 2021-05-31 10:13 ` Janosch Frank
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).