Re: [PATCH] tools/xenstored: Fix off-by-one in dump_state_nodes() [and 3 more messages]

2021-07-30 Thread Ian Jackson
Julien Grall writes ("[PATCH] tools/xenstored: Fix off-by-one in dump_state_nodes()"): > The maximum path length supported by Xenstored protocol is > XENSTORE_ABS_PATH_MAX (i.e 3072). This doesn't take into account the > NUL at the end of the path. ... Julien Gral

Re: [PATCH] tools/xenstored: Fix off-by-one in dump_state_nodes()

2021-07-30 Thread Juergen Gross
On 29.07.21 11:34, Julien Grall wrote: From: Julien Grall The maximum path length supported by Xenstored protocol is XENSTORE_ABS_PATH_MAX (i.e 3072). This doesn't take into account the NUL at the end of the path. However, the code to dump the nodes will allocate a buffer of XENSTORE_ABS_PATH.

[PATCH] tools/xenstored: Fix off-by-one in dump_state_nodes()

2021-07-29 Thread Julien Grall
From: Julien Grall The maximum path length supported by Xenstored protocol is XENSTORE_ABS_PATH_MAX (i.e 3072). This doesn't take into account the NUL at the end of the path. However, the code to dump the nodes will allocate a buffer of XENSTORE_ABS_PATH. As a result it may not be possible to li