xen/hypfs: fix writing of custom parameter
authorJuergen Gross <jgross@suse.com>
Fri, 11 Sep 2020 12:20:10 +0000 (14:20 +0200)
committerJan Beulich <jbeulich@suse.com>
Fri, 11 Sep 2020 12:20:10 +0000 (14:20 +0200)
commitb4e41b1750d550bf2b1ccf97ee46f4f682bdbb62
tree37f1aab3211dae041dfece7d52dfe1421af2f472
parent0fcfe9d95f8bb1d53cf68038f13bc3563020e8cd
xen/hypfs: fix writing of custom parameter

Today the maximum allowed data length for writing a hypfs node is
tested in the generic hypfs_write() function. For custom runtime
parameters this might be wrong, as the maximum allowed size is derived
from the buffer holding the current setting, while there might be ways
to set the parameter needing more characters than the minimal
representation of that value.

One example for this is the "ept" parameter. Its value buffer is sized
to be able to hold the string "exec-sp=0" or "exec-sp=1", while it is
allowed to use e.g. "no-exec-sp" or "exec-sp=yes" for setting it.

Fix that by moving the length check one level down to the type
specific write function.

In order to avoid allocation of arbitrary sized buffers use a new
MAX_PARAM_SIZE macro as an upper limit for custom writes. The value
of MAX_PARAM_SIZE is the same as the limit in parse_params() for a
single parameter.

Fixes: a659d7cab9af ("xen: add runtime parameter access support to hypfs")
Reported-by: Andrew Cooper <andrew.cooper3@citrix.com>
Signed-off-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Tested-by: Andrew Cooper <andrew.cooper3@citrix.com>
xen/common/hypfs.c
xen/common/kernel.c
xen/include/xen/param.h