Re: [Xen-devel] [PATCH 1/1] tools/livepatch: cleanup unnecessary "j = ARRAY_SIZE(action_options); "
On Tue, Jun 14, 2016 at 09:22:45PM +0200, Olaf Hering wrote: > On Fri, Jun 10, Dongli Zhang wrote: > > > Local variable "j" would be used only when "i == ARRAY_SIZE(main_options)" > > is true. Thus, it is not necessary to update "j" when "i == > > ARRAY_SIZE(main_options)" is false. > > This breaks the build with gcc45: > > [ 153s] cc1: warnings being treated as errors > [ 153s] xen-livepatch.c: In function 'main': > [ 153s] xen-livepatch.c:415:12: error: 'j' may be used uninitialized in this > function > [ 153s] make[3]: *** [xen-livepatch.o] Error 1 > > Olaf ? diff --git a/tools/misc/xen-livepatch.c b/tools/misc/xen-livepatch.c index 3162489..62c072e 100644 --- a/tools/misc/xen-livepatch.c +++ b/tools/misc/xen-livepatch.c @@ -412,7 +412,7 @@ struct { int main(int argc, char *argv[]) { -int i, j, ret; +int i, j = 0, ret; if ( argc <= 1 ) { ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH 1/1] tools/livepatch: cleanup unnecessary "j = ARRAY_SIZE(action_options); "
On Fri, Jun 10, Dongli Zhang wrote: > Local variable "j" would be used only when "i == ARRAY_SIZE(main_options)" > is true. Thus, it is not necessary to update "j" when "i == > ARRAY_SIZE(main_options)" is false. This breaks the build with gcc45: [ 153s] cc1: warnings being treated as errors [ 153s] xen-livepatch.c: In function 'main': [ 153s] xen-livepatch.c:415:12: error: 'j' may be used uninitialized in this function [ 153s] make[3]: *** [xen-livepatch.o] Error 1 Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH 1/1] tools/livepatch: cleanup unnecessary "j = ARRAY_SIZE(action_options); "
On Fri, Jun 10, 2016 at 12:02:52AM +0800, Dongli Zhang wrote: > Local variable "j" would be used only when "i == ARRAY_SIZE(main_options)" > is true. Thus, it is not necessary to update "j" when "i == > ARRAY_SIZE(main_options)" is false. > > Signed-off-by: Dongli Zhang> Reviewed-by: Konrad Rzeszutek Wilk Thanks, I've queued this up in my next batch. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
[Xen-devel] [PATCH 1/1] tools/livepatch: cleanup unnecessary "j = ARRAY_SIZE(action_options); "
Local variable "j" would be used only when "i == ARRAY_SIZE(main_options)" is true. Thus, it is not necessary to update "j" when "i == ARRAY_SIZE(main_options)" is false. Signed-off-by: Dongli ZhangReviewed-by: Konrad Rzeszutek Wilk --- tools/misc/xen-livepatch.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tools/misc/xen-livepatch.c b/tools/misc/xen-livepatch.c index 28f339a..3162489 100644 --- a/tools/misc/xen-livepatch.c +++ b/tools/misc/xen-livepatch.c @@ -435,8 +435,7 @@ int main(int argc, char *argv[]) "'xen-livepatch help'\n", argv[1]); return 1; } -} else -j = ARRAY_SIZE(action_options); +} xch = xc_interface_open(0,0,0); if ( !xch ) -- 1.9.1 ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel