Name and description in struct rv_monitor are currently defined as pointers, which are assigned during initialisation. This means they are not clearly in the structure and this can be a problem if the struct needs to be reused in other contexts (like BPF's struct_opts).
Make the fields static arrays with defined size, limit the description to 128 bytes (much longer than any current description). This slightly increases the kernel size as the full space for the strings is reserved, but that's a minor increase as each monitor has a single instance of struct rv_monitor, total increase around 100B per monitor, uncompressed. Signed-off-by: Gabriele Monaco <[email protected]> --- include/linux/rv.h | 9 ++++++--- kernel/trace/rv/rv.c | 10 +++++----- kernel/trace/rv/rv.h | 3 --- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/include/linux/rv.h b/include/linux/rv.h index 541ba404926a..a3a332b49747 100644 --- a/include/linux/rv.h +++ b/include/linux/rv.h @@ -7,7 +7,10 @@ #ifndef _LINUX_RV_H #define _LINUX_RV_H -#define MAX_DA_NAME_LEN 32 +#define MAX_RV_MONITOR_NAME_SIZE 32 +#define MAX_RV_REACTOR_NAME_SIZE 32 +#define MAX_RV_DESCRIPTION_SIZE 128 + #define MAX_DA_RETRY_RACING_EVENTS 3 #define RV_MON_GLOBAL 0 @@ -137,8 +140,8 @@ struct rv_reactor { #endif struct rv_monitor { - const char *name; - const char *description; + char name[MAX_RV_MONITOR_NAME_SIZE]; + char description[MAX_RV_DESCRIPTION_SIZE]; bool enabled; int (*enable)(void); void (*disable)(void); diff --git a/kernel/trace/rv/rv.c b/kernel/trace/rv/rv.c index 29f155c6968b..f77c5865f41a 100644 --- a/kernel/trace/rv/rv.c +++ b/kernel/trace/rv/rv.c @@ -392,11 +392,11 @@ static ssize_t monitor_desc_read_data(struct file *filp, char __user *user_buf, loff_t *ppos) { struct rv_monitor *mon = filp->private_data; - char buff[256]; + char buff[MAX_RV_DESCRIPTION_SIZE + 2]; memset(buff, 0, sizeof(buff)); - snprintf(buff, sizeof(buff), "%s\n", mon->description); + snprintf(buff, sizeof(buff), "%.*s\n", MAX_RV_DESCRIPTION_SIZE, mon->description); return simple_read_from_buffer(user_buf, count, ppos, buff, strlen(buff) + 1); } @@ -756,9 +756,9 @@ int rv_register_monitor(struct rv_monitor *monitor, struct rv_monitor *parent) struct rv_monitor *r; int retval = 0; - if (strlen(monitor->name) >= MAX_RV_MONITOR_NAME_SIZE) { - pr_info("Monitor %s has a name longer than %d\n", monitor->name, - MAX_RV_MONITOR_NAME_SIZE); + if (strnlen(monitor->name, MAX_RV_MONITOR_NAME_SIZE) == MAX_RV_MONITOR_NAME_SIZE) { + pr_info("Monitor %.*s has a name longer than %d\n", MAX_RV_MONITOR_NAME_SIZE, + monitor->name, MAX_RV_MONITOR_NAME_SIZE); return -EINVAL; } diff --git a/kernel/trace/rv/rv.h b/kernel/trace/rv/rv.h index 2c0f51ff9d5c..07512046e5cc 100644 --- a/kernel/trace/rv/rv.h +++ b/kernel/trace/rv/rv.h @@ -19,9 +19,6 @@ struct rv_interface { DEFINE_FREE(rv_remove, struct dentry *, if (_T) rv_remove(_T)); -#define MAX_RV_MONITOR_NAME_SIZE 32 -#define MAX_RV_REACTOR_NAME_SIZE 32 - extern struct mutex rv_interface_lock; extern struct list_head rv_monitors_list; -- 2.55.0
