From: Ian Jackson <iwj@xenproject.org>
To: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: Xen-devel <xen-devel@lists.xenproject.org>, Wei Liu <wl@xen.org>,
Juergen Gross <jgross@suse.com>
Subject: Re: [PATCH 06/10] stubdom/xenstored: Fix uninitialised variables in lu_read_state()
Date: Tue, 16 Feb 2021 16:10:41 +0000 [thread overview]
Message-ID: <24619.61057.971290.452710@mariner.uk.xensource.com> (raw)
In-Reply-To: <20210212153953.4582-7-andrew.cooper3@citrix.com>
Andrew Cooper writes ("[PATCH 06/10] stubdom/xenstored: Fix uninitialised variables in lu_read_state()"):
> Various version of gcc, when compiling with -Og, complain:
>
> xenstored_control.c: In function ‘lu_read_state’:
> xenstored_control.c:540:11: error: ‘state.size’ is used uninitialized in this
> function [-Werror=uninitialized]
> if (state.size == 0)
> ~~~~~^~~~~
> xenstored_control.c:543:6: error: ‘state.buf’ may be used uninitialized in
> this function [-Werror=maybe-uninitialized]
> pre = state.buf;
> ~~~~^~~~~~~~~~~
> xenstored_control.c:550:23: error: ‘state.buf’ may be used uninitialized in
> this function [-Werror=maybe-uninitialized]
> (void *)head - state.buf < state.size;
> ~~~~~^~~~
> xenstored_control.c:550:35: error: ‘state.size’ may be used uninitialized in
> this function [-Werror=maybe-uninitialized]
> (void *)head - state.buf < state.size;
> ~~~~~^~~~~
>
> Interestingly, this is only in the stubdom build. I can't identify any
> relevant differences vs the regular tools build.
#ifdef __MINIOS__
static void lu_get_dump_state(struct lu_dump_state *state)
{
}
So the compiler is right to complain that
lu_get_dump_state(&state);
if (state.size == 0)
barf_perror("No state found after live-update");
this will use state.size uninitialised.
It's probably just luck that this works at all, if it does,
anywhere...
Reviewed-by: Ian Jackson <iwj@xenproject.org>
Release-Acked-by: Ian Jackson <iwj@xenproject.org>
Ian.
next prev parent reply other threads:[~2021-02-16 16:10 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-12 15:39 [PATCH for-4.15 00/10] tools: Support to use abi-dumper on libraries Andrew Cooper
2021-02-12 15:39 ` [PATCH 01/10] tools/xl: Fix exit code for `xl vkbattach` Andrew Cooper
2021-02-16 15:57 ` Ian Jackson
2021-02-16 16:25 ` Ian Jackson
2021-02-12 15:39 ` [PATCH 02/10] tools/libxg: Fix uninitialised variable in write_x86_cpu_policy_records() Andrew Cooper
2021-02-12 15:54 ` Jan Beulich
2021-02-16 15:57 ` Ian Jackson
2021-02-12 15:39 ` [PATCH 03/10] tools/libxg: Fix uninitialised variable in meminit() Andrew Cooper
2021-02-12 15:55 ` Julien Grall
2021-02-12 19:35 ` Andrew Cooper
2021-02-12 20:01 ` [PATCH v1.1 03/10] tools/libxg: Drop stale p2m logic from ARM's meminit() Andrew Cooper
2021-02-16 16:00 ` Ian Jackson
2021-02-16 16:27 ` Julien Grall
2021-02-12 15:39 ` [PATCH 04/10] tools/libxl: Fix uninitialised variable in libxl__domain_get_device_model_uid() Andrew Cooper
2021-02-16 16:07 ` Ian Jackson
2021-02-16 17:43 ` [PATCH v2 " Andrew Cooper
2021-02-16 17:57 ` Ian Jackson
2021-02-16 18:05 ` Ian Jackson
2021-02-12 15:39 ` [PATCH 05/10] tools/libxl: Fix uninitialised variable in libxl__write_stub_dmargs() Andrew Cooper
2021-02-16 16:07 ` Ian Jackson
2021-02-16 16:26 ` Ian Jackson
2021-02-12 15:39 ` [PATCH 06/10] stubdom/xenstored: Fix uninitialised variables in lu_read_state() Andrew Cooper
2021-02-12 16:08 ` Jürgen Groß
2021-02-12 17:01 ` Andrew Cooper
2021-02-13 9:36 ` Jürgen Groß
2021-02-15 14:12 ` Andrew Cooper
2021-02-15 15:36 ` [PATCH v1.1 " Andrew Cooper
2021-02-16 16:10 ` Ian Jackson [this message]
2021-02-12 15:39 ` [PATCH 07/10] tools: Use -Og for debug builds when available Andrew Cooper
2021-02-12 16:04 ` Jan Beulich
2021-02-12 16:09 ` Andrew Cooper
2021-02-12 16:14 ` Jan Beulich
2021-02-16 16:26 ` Ian Jackson
2021-02-16 16:11 ` Ian Jackson
2021-02-12 15:39 ` [PATCH 08/10] tools: Check for abi-dumper in ./configure Andrew Cooper
2021-02-16 16:11 ` Ian Jackson
2021-02-16 16:27 ` Ian Jackson
2021-02-12 15:39 ` [PATCH 09/10] tools/libs: Add rule to generate headers.lst Andrew Cooper
2021-02-16 16:13 ` Ian Jackson
2021-02-16 16:48 ` [PATCH v2 " Andrew Cooper
2021-02-16 17:32 ` Ian Jackson
2021-02-12 15:39 ` [PATCH 10/10] tools/libs: Write out an ABI analysis when abi-dumper is available Andrew Cooper
2021-02-12 16:12 ` Jan Beulich
2021-02-12 17:03 ` Andrew Cooper
2021-02-12 18:01 ` [PATCH v1.1 " Andrew Cooper
2021-02-16 16:15 ` [PATCH " Ian Jackson
2021-02-16 16:30 ` Ian Jackson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=24619.61057.971290.452710@mariner.uk.xensource.com \
--to=iwj@xenproject.org \
--cc=andrew.cooper3@citrix.com \
--cc=jgross@suse.com \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).