diff mbox

[16/18] Increase host buffer size

Message ID 1444333491-16265-17-git-send-email-bmarzins@redhat.com (mailing list archive)
State Not Applicable, archived
Delegated to: Benjamin Marzinski
Headers show

Commit Message

Benjamin Marzinski Oct. 8, 2015, 7:44 p.m. UTC
Currently the host buffer only has space for 7 characters, this means
on systems with many scsi hosts (1000 or more), multipath will overflow
this buffer. This can happen pretty easily if there are a large number
of iscsi devices.  This patch increases the host buffer to hold 15
characters, which can deal with 100000000000000 scsi hosts.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
---
 libmultipath/structs.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Hannes Reinecke Oct. 12, 2015, 7:10 a.m. UTC | #1
On 10/08/2015 09:44 PM, Benjamin Marzinski wrote:
> Currently the host buffer only has space for 7 characters, this means
> on systems with many scsi hosts (1000 or more), multipath will overflow
> this buffer. This can happen pretty easily if there are a large number
> of iscsi devices.  This patch increases the host buffer to hold 15
> characters, which can deal with 100000000000000 scsi hosts.
> 
> Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
> ---
>  libmultipath/structs.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libmultipath/structs.h b/libmultipath/structs.h
> index 85a8fdc..99b6aae 100644
> --- a/libmultipath/structs.h
> +++ b/libmultipath/structs.h
> @@ -15,7 +15,7 @@
>  #define BLK_DEV_SIZE		33
>  #define PATH_SIZE		512
>  #define NAME_SIZE		512
> -#define HOST_NAME_LEN		8
> +#define HOST_NAME_LEN		16
>  #define SLOT_NAME_SIZE		40
>  
>  #define SCSI_VENDOR_SIZE	9
> 
Reviewed-by: Hannes Reinecke <hare@suse.de>

Cheers,

Hannes
diff mbox

Patch

diff --git a/libmultipath/structs.h b/libmultipath/structs.h
index 85a8fdc..99b6aae 100644
--- a/libmultipath/structs.h
+++ b/libmultipath/structs.h
@@ -15,7 +15,7 @@ 
 #define BLK_DEV_SIZE		33
 #define PATH_SIZE		512
 #define NAME_SIZE		512
-#define HOST_NAME_LEN		8
+#define HOST_NAME_LEN		16
 #define SLOT_NAME_SIZE		40
 
 #define SCSI_VENDOR_SIZE	9