multipath-tools: fix the bug while processing vpd 0x83 designation descriptors
authorMoger, Babu <Babu.Moger@netapp.com>
Wed, 22 Feb 2012 18:09:00 +0000 (18:09 +0000)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Wed, 22 Feb 2012 21:45:36 +0000 (22:45 +0100)
This patch fixes the bug while processing the vpd 0x83 designation descriptors.
Removing the buggy check(> sizeof(buf))while loping the descriptors. Sizeof(buf) will
always return 8 (in 64 bit machine). Descriptor length can be more than 8 bytes in
some cases. This was causing problems while configuring my storage with alua.

Signed-off-by: Babu Moger <babu.moger@netapp.com>
libmultipath/prioritizers/alua_rtpg.c

index f98ae9e..b76db72 100644 (file)
@@ -184,9 +184,6 @@ get_target_port_group(int fd)
 
                rc = -RTPG_NO_TPG_IDENTIFIER;
                FOR_EACH_VPD83_DSCR(vpd83, dscr) {
-                       if ((((char *) dscr) - ((char *) vpd83)) > sizeof(buf))
-                               break;
-
                        if (vpd83_dscr_istype(dscr, IDTYPE_TARGET_PORT_GROUP)) {
                                struct vpd83_tpg_dscr * p;