[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH] tools/xl: fix uninitialized variable in xl_vdispl
On 03/13/2018 04:43 AM, Doug Goldstein wrote: > The code added in 7a48622a78a0b452e8afa55b8442c958abd226a7 could use rc > uninitialized in main_vdisplattach(). > > Signed-off-by: Doug Goldstein <cardoe@xxxxxxxxxx> It looks like that was designed on purpose to use the uninitialized warnings to catch paths where the rc wasn't specifically set. The only path where it's not set is if dryrun_only is true; and in that case, we probably actually want it to succeed, not fail. No matter what, rc = 0 should be added to the dryrun_only path. If we want to make ERROR_FAIL the default, then the various places rc is set to ERROR_FAIL should be removed. -George _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |