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

Re: [win-pv-devel] [PATCH] Increase FrontendPath length for long DeviceIds


  • To: Owen Smith <owen.smith@xxxxxxxxxx>, "win-pv-devel@xxxxxxxxxxxxxxxxxxxx" <win-pv-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: "Durrant, Paul" <pdurrant@xxxxxxxxxxxx>
  • Date: Wed, 15 Jan 2020 12:09:30 +0000
  • Accept-language: en-GB, en-US
  • Delivery-date: Wed, 15 Jan 2020 12:16:23 +0000
  • Ironport-sdr: tcwM73ePtwwRO5fFefoZzoI18paK5fskqQ7g7VARVCR7MIkS6ZfRm7h9STcAVwwGNNRoOjgvDf 1rSq5hstMIJQ==
  • List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>
  • Thread-index: AQHVy5o844Iy8LEJhEqdWVzV33s+2qfrofGw
  • Thread-topic: [win-pv-devel] [PATCH] Increase FrontendPath length for long DeviceIds

> -----Original Message-----
> From: win-pv-devel <win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx> On Behalf
> Of Owen Smith
> Sent: 15 January 2020 12:51
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Owen Smith <owen.smith@xxxxxxxxxx>
> Subject: [win-pv-devel] [PATCH] Increase FrontendPath length for long
> DeviceIds
> 
> The DeviceIds maximum value is 1 << 28 | 0xfffff << 8 | 0xff, which is a
> 9 digit decimal numver. Values higher than this are invalid. Ensure the
> FrontendPath buffer is large enough to contain valid DeviceId values.
> 
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>

Acked-by: Paul Durrant <paul@xxxxxxx>

> ---
>  src/xenvbd/frontend.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/xenvbd/frontend.c b/src/xenvbd/frontend.c
> index 3fa19a0..d74f0b3 100644
> --- a/src/xenvbd/frontend.c
> +++ b/src/xenvbd/frontend.c
> @@ -62,7 +62,7 @@ struct _XENVBD_FRONTEND {
>      PXENVBD_TARGET              Target;
>      ULONG                       TargetId;
>      ULONG                       DeviceId;
> -    CHAR
> FrontendPath[sizeof("device/vbd/XXXXXXXX")];
> +    CHAR
> FrontendPath[sizeof("device/vbd/XXXXXXXXX")];
>      PCHAR                       BackendPath;
>      CHAR
> TargetPath[sizeof("data/scsi/target/XXXX")];
>      USHORT                      BackendDomain;
> --
> 2.16.2.windows.1
> 
> 
> _______________________________________________
> win-pv-devel mailing list
> win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> https://lists.xenproject.org/mailman/listinfo/win-pv-devel
_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/win-pv-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.