[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [RESEND PATCH] monitor: properly handle illegal fd/vhos
From: |
Luiz Capitulino |
Subject: |
[Qemu-devel] Re: [RESEND PATCH] monitor: properly handle illegal fd/vhostfd from command line |
Date: |
Fri, 22 Oct 2010 10:44:15 -0200 |
On Thu, 21 Oct 2010 19:29:02 +0800
Jason Wang <address@hidden> wrote:
> When hanlding fd/vhostfd form command line through net_handle_fd_param(),
> we need to check mon and return value of strtol() other than we could
> get segmentation fault or invalid fd when user type an illegal fd/vhostfd.
>
> This patch is based on the suggestions from
> Luiz Capitulino <address@hidden>.
>
> Signed-off-by: Jason Wang <address@hidden>
> ---
> net.c | 12 +++++++++---
> 1 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/net.c b/net.c
> index ed74c7f..ab9c3bb 100644
> --- a/net.c
> +++ b/net.c
> @@ -774,8 +774,8 @@ int qemu_find_nic_model(NICInfo *nd, const char * const
> *models,
>
> int net_handle_fd_param(Monitor *mon, const char *param)
> {
> - if (!qemu_isdigit(param[0])) {
> - int fd;
> + int fd;
Better to add a space here.
> + if (!qemu_isdigit(param[0]) && mon) {
>
> fd = monitor_get_fd(mon, param);
> if (fd == -1) {
> @@ -785,7 +785,13 @@ int net_handle_fd_param(Monitor *mon, const char *param)
>
> return fd;
> } else {
> - return strtol(param, NULL, 0);
> + char *endptr = NULL;
> +
> + fd = strtol(param, &endptr, 10);
> + if (*endptr || (fd == 0 && param == endptr)) {
> + return -1;
> + }
> + return fd;
> }
You can put 'return fd' here and drop the two above, also this is not
a monitor patch, it's a networking one.
Otherwise looks ok to me.
> }
>
>