Re: [PATCH 01/15] Return correct error and len in response to invalid tgtadm requests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 09 Jun 2009 18:21:48 +0200
Arne Redlich <arne.redlich@xxxxxxxxxxxxxx> wrote:

> Signed-off-by: Arne Redlich <arne.redlich@xxxxxxxxxxxxxx>
> ---
>  usr/mgmt.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/usr/mgmt.c b/usr/mgmt.c
> index 5332dc0..7fc5d29 100644
> --- a/usr/mgmt.c
> +++ b/usr/mgmt.c
> @@ -359,6 +359,9 @@ static int tgt_mgmt(struct mgmt_task *mtask)
>  							mtask->buf, len);
>  
>  			set_show_results(rsp, &err);
> +		} else {
> +			rsp->err = err;
> +			rsp->len = sizeof(*rsp);
>  		}
>  		break;
>  	}
> -- 
> 1.6.0.4

Merged, thanks!

We really need to clean up the error value handling (especially in
mgmt.c).
--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux