| Message ID | 20210225174131.10115-1-julien@xen.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | xenstore: Address coverity issues in the LiveUpdate code | expand |
Julien Grall writes ("[PATCH for-4.15 0/5] xenstore: Address coverity issues in the LiveUpdate code"):
> tools/xenstored: Avoid unnecessary talloc_strdup() in do_control_lu()
> tools/xenstored: Avoid unnecessary talloc_strdup() in do_lu_start()
> tools/xenstored: control: Store the save filename in lu_dump_state
> tools/xenstore-control: Don't leak buf in live_update_start()
These four are actual bugfixes:
Release-Acked-by: Ian Jackson <iwj@xenproject.org>
> tools/xenstored: Silence coverity when using xs_state_* structures
For this I can't see a reason to give a release ack ? See also Andy's
comments.
Ian.
Hi Ian, On 25/02/2021 17:54, Ian Jackson wrote: > Julien Grall writes ("[PATCH for-4.15 0/5] xenstore: Address coverity issues in the LiveUpdate code"): >> tools/xenstored: Avoid unnecessary talloc_strdup() in do_control_lu() >> tools/xenstored: Avoid unnecessary talloc_strdup() in do_lu_start() >> tools/xenstored: control: Store the save filename in lu_dump_state >> tools/xenstore-control: Don't leak buf in live_update_start() > > These four are actual bugfixes: > > Release-Acked-by: Ian Jackson <iwj@xenproject.org> Thanks! > >> tools/xenstored: Silence coverity when using xs_state_* structures > > For this I can't see a reason to give a release ack ? See also Andy's > comments. I don't have a reason for this one as it is so far just silencing Coverity. Sorry I should have mention that this one is not really 4.15 material. Cheers,
Julien Grall writes ("Re: [PATCH for-4.15 0/5] xenstore: Address coverity issues in the LiveUpdate code"):
> On 25/02/2021 17:54, Ian Jackson wrote:
> > Julien Grall writes ("[PATCH for-4.15 0/5] xenstore: Address coverity issues in the LiveUpdate code"):
> >> tools/xenstored: Silence coverity when using xs_state_* structures
> >
> > For this I can't see a reason to give a release ack ? See also Andy's
> > comments.
>
> I don't have a reason for this one as it is so far just silencing
> Coverity. Sorry I should have mention that this one is not really 4.15
> material.
No problem, thanks for the fixes!
Ian.
From: Julien Grall <jgrall@amazon.com> Hi all, The AWS coverity instance spotted a few issues that could either leak memory and derefence NULL pointer. All the patches are candidate for 4.15 as they are hardening XenStored code. The changes are low risks. Cheers, Julien Grall (5): tools/xenstored: Avoid unnecessary talloc_strdup() in do_control_lu() tools/xenstored: Avoid unnecessary talloc_strdup() in do_lu_start() tools/xenstored: control: Store the save filename in lu_dump_state tools/xenstore-control: Don't leak buf in live_update_start() tools/xenstored: Silence coverity when using xs_state_* structures tools/xenstore/include/xenstore_state.h | 6 +++--- tools/xenstore/xenstore_control.c | 4 +++- tools/xenstore/xenstored_control.c | 26 +++++++++++-------------- 3 files changed, 17 insertions(+), 19 deletions(-)