<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 20, 2019 at 2:51 PM Jan Stancek <<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
----- Original Message -----<br>
> On Tue, Nov 19, 2019 at 6:29 PM Jan Stancek <<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>> wrote:<br>
> <br>
> > Rachel reports, that pivot_root01 crashes on latest LTP:<br>
> >   Thread 3.1 "pivot_root01" received signal SIGSEGV, Segmentation fault.<br>
> >   0x00000000004062c4 in safe_mount (file=file@entry=0x413017<br>
> > "pivot_root01.c"<br>
> >   733             if (strcmp(filesystemtype, "ntfs")) {<br>
> ><br>
> > Don't pass NULL ptr to strcmp. Also fix return in unreachable path,<br>
> > to suppress warning about returning uninitialized 'rval'.<br>
> ><br>
> <br>
> Good catch.<br>
> <br>
> And shouldn't we give a "correct" fs_type in pivot_root01.c too? Otherwise,<br>
> the test would be failed on an invalid mounted operation I guess.<br>
<br>
Per man-page, it should be ignored.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Ok, then this patch looks good.</div><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>