<font size=2 face="sans-serif">Hello Christophe,</font>
<br>
<br><font size=2 face="sans-serif">This issue </font><font size=2>affects
the accuracy of regular matching,</font>
<br><font size=2 face="sans-serif">and the patch bellow had been reviewed
by Hannes,</font>
<br><font size=2 face="sans-serif">Can you merge it to Opensvc?</font>
<br>
<br><font size=2 face="sans-serif">Thanks</font>
<br><font size=2 face="sans-serif">Tang</font>
<br>
<br>
<br><font size=2 face="sans-serif">On 10/14/2016 04:03 AM, huang.wei56@zte.com.cn
wrote:</font>
<br><font size=2 face="sans-serif">> From: "wei.huang" <huang.wei56@zte.com.cn></font>
<br><font size=2 face="sans-serif">> </font>
<br><font size=2 face="sans-serif">> Problem:</font>
<br><font size=2 face="sans-serif">> when we configure a device like
vendor, product, revision all null in multipath.conf, hwe_regmatch always
return 0.</font>
<br><font size=2 face="sans-serif">> </font>
<br><font size=2 face="sans-serif">> Reasons:</font>
<br><font size=2 face="sans-serif">> \!hwe2->vendor, \!hwe2->product
and \!hwe2->revision are all true.</font>
<br><font size=2 face="sans-serif">> </font>
<br><font size=2 face="sans-serif">> Signed-off-by: wei.huang <huang.wei56@zte.com.cn></font>
<br><font size=2 face="sans-serif">> ---</font>
<br><font size=2 face="sans-serif">>  libmultipath/config.c | 3
++-</font>
<br><font size=2 face="sans-serif">>  1 file changed, 2 insertions(+),
1 deletion(-)</font>
<br><font size=2 face="sans-serif">> </font>
<br><font size=2 face="sans-serif">> diff --git a/libmultipath/config.c
b/libmultipath/config.c</font>
<br><font size=2 face="sans-serif">> index a48b8af..d99cd75 100644</font>
<br><font size=2 face="sans-serif">> --- a/libmultipath/config.c</font>
<br><font size=2 face="sans-serif">> +++ b/libmultipath/config.c</font>
<br><font size=2 face="sans-serif">> @@ -80,7 +80,8 @@ hwe_regmatch
(struct hwentry *hwe1, struct hwentry *hwe2)</font>
<br><font size=2 face="sans-serif">>          
            regcomp(&rre, hwe1->revision,
REG_EXTENDED|REG_NOSUB))</font>
<br><font size=2 face="sans-serif">>          
                 
       goto out_pre;</font>
<br><font size=2 face="sans-serif">>  </font>
<br><font size=2 face="sans-serif">> -        
        if ((!hwe1->vendor || !hwe2->vendor
||</font>
<br><font size=2 face="sans-serif">> +        
        if ((hwe2->vendor || hwe2->product
|| hwe2->revision) &&</font>
<br><font size=2 face="sans-serif">> +        
            (!hwe1->vendor ||
!hwe2->vendor ||</font>
<br><font size=2 face="sans-serif">>          
             !regexec(&vre,
hwe2->vendor, 0, NULL, 0)) &&</font>
<br><font size=2 face="sans-serif">>          
            (!hwe1->product ||
!hwe2->product ||</font>
<br><font size=2 face="sans-serif">>          
             !regexec(&pre,
hwe2->product, 0, NULL, 0)) &&</font>
<br><font size=2 face="sans-serif">> </font>
<br><font size=2 face="sans-serif">Good point.</font>
<br>
<br><font size=2 face="sans-serif">Reviewed-by: Hannes Reinecke <hare@suse.com></font>
<br>
<br><font size=2 face="sans-serif">Cheers,</font>
<br>
<br><font size=2 face="sans-serif">Hannes</font>
<br><font size=2 face="sans-serif">-- </font>
<br><font size=2 face="sans-serif">Dr. Hannes Reinecke      
               
              Teamlead
Storage & Networking</font>
<br><font size=2 face="sans-serif">hare@suse.de        
                 
               
                 
     +49 911 74053 688</font>
<br><font size=2 face="sans-serif">SUSE LINUX GmbH, Maxfeldstr. 5, 90409
Nščrnberg</font>
<br><font size=2 face="sans-serif">GF: F. Imend?rffer, J. Smithard, J.
Guild, D. Upmanyu, G. Norton</font>
<br><font size=2 face="sans-serif">HRB 21284 (AG Nščrnberg)</font>