<div dir="ltr">Merged.<div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 1, 2016 at 5:05 PM, Benjamin Marzinski <span dir="ltr"><<a href="mailto:bmarzins@redhat.com" target="_blank">bmarzins@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Nov 01, 2016 at 11:12:46AM +0800, <a href="mailto:tang.junhui@zte.com.cn">tang.junhui@zte.com.cn</a> wrote:<br>
>    Hello Christophe,<br>
<br>
ACK<br>
<br>
-Ben<br>
<div><div class="h5"><br>
><br>
>    This issue affects the accuracy of regular matching,<br>
>    and the patch bellow had been reviewed by Hannes,<br>
>    Can you merge it to Opensvc?<br>
><br>
>    Thanks<br>
>    Tang<br>
><br>
>    On 10/14/2016 04:03 AM, <a href="mailto:huang.wei56@zte.com.cn">huang.wei56@zte.com.cn</a> wrote:<br>
>    > From: "wei.huang" <<a href="mailto:huang.wei56@zte.com.cn">huang.wei56@zte.com.cn</a>><br>
>    ><br>
>    > Problem:<br>
>    > when we configure a device like vendor, product, revision all null in<br>
>    multipath.conf, hwe_regmatch always return 0.<br>
>    ><br>
>    > Reasons:<br>
>    > \!hwe2->vendor, \!hwe2->product and \!hwe2->revision are all true.<br>
>    ><br>
>    > Signed-off-by: wei.huang <<a href="mailto:huang.wei56@zte.com.cn">huang.wei56@zte.com.cn</a>><br>
>    > ---<br>
>    >  libmultipath/config.c | 3 ++-<br>
>    >  1 file changed, 2 insertions(+), 1 deletion(-)<br>
>    ><br>
>    > diff --git a/libmultipath/config.c b/libmultipath/config.c<br>
>    > index a48b8af..d99cd75 100644<br>
>    > --- a/libmultipath/config.c<br>
>    > +++ b/libmultipath/config.c<br>
>    > @@ -80,7 +80,8 @@ hwe_regmatch (struct hwentry *hwe1, struct hwentry<br>
>    *hwe2)<br>
>    >                       regcomp(&rre, hwe1->revision,<br>
>    REG_EXTENDED|REG_NOSUB))<br>
>    >                                    goto out_pre;<br>
>    >  <br>
>    > -                 if ((!hwe1->vendor || !hwe2->vendor ||<br>
>    > +                 if ((hwe2->vendor || hwe2->product || hwe2->revision)<br>
>    &&<br>
>    > +                     (!hwe1->vendor || !hwe2->vendor ||<br>
>    >                        !regexec(&vre, hwe2->vendor, 0, NULL, 0)) &&<br>
>    >                       (!hwe1->product || !hwe2->product ||<br>
>    >                        !regexec(&pre, hwe2->product, 0, NULL, 0)) &&<br>
>    ><br>
>    Good point.<br>
><br>
>    Reviewed-by: Hannes Reinecke <<a href="mailto:hare@suse.com">hare@suse.com</a>><br>
><br>
>    Cheers,<br>
><br>
>    Hannes<br>
>    --<br>
>    Dr. Hannes Reinecke                                     Teamlead Storage &<br>
>    Networking<br>
>    <a href="mailto:hare@suse.de">hare@suse.de</a>                                                              <br>
>       <a href="tel:%2B49%20911%2074053%20688" value="+4991174053688">+49 911 74053 688</a><br>
</div></div>>    SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N��rnberg<br>
<span class="">>    GF: F. Imend?rffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton<br>
</span>>    HRB 21284 (AG N��rnberg)<br>
</blockquote></div><br></div>