<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,sans-serif">On Tue, Nov 10, 2015 at 9:35 PM, Eric Sandeen </span><span dir="ltr" style="font-family:arial,sans-serif"><<a href="mailto:sandeen@redhat.com" target="_blank">sandeen@redhat.com</a>></span><span style="font-family:arial,sans-serif"> wrote:</span><br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">> On Tue, Nov 10, 2015 at 12:21:10PM -0600, Eric Sandeen wrote:<br>
>> Commit:<br>
>><br>
>> 7141fc xfsprogs: make fsr use mntinfo when there is no mntent<br>
>><br>
>> added an inadvertent "break;" to initallfs() after the call<br>
>> to find_mountpoint_check(); this is likely a cut & paste error<br>
>> from the call in find_mountpoint(), where we really *do* want to<br>
>> stop after the first one we find.<br>
>><br>
>> Fix that by removing the break, and fix the declaration-after-code.<br>
><br>
> That's not the right fix - the find_mountpoint_check() shoul dnot be<br>
> there at all. Patch to clean it all up is below.<br>
><br>
> -Dave.<br>
> --<br>
> Dave Chinner<br>
> <a href="mailto:david@fromorbit.com">david@fromorbit.com</a><br>
><br>
> fsr: clean up mountpoint checks<br>
><br>
> From: Dave Chinner <<a href="mailto:dchinner@redhat.com">dchinner@redhat.com</a>><br>
><br>
> Fix up a stray hunk of code from commit 7141fc5 ("xfsprogs: make fsr<br>
> use mntinfo when there is no mntent") that coverity reported. Also<br>
> clean up a couple of whitespace issues introduced with that  commit,<br>
> too.<br>
<br>
</span>Um, ok, well it wasn't your misapplication, it was there in the original patch,<br>
and I wasn't sure of Jan's intent, having gone so far as to add local vars<br>
to support that placement.  ;)<br>
<br>
This seems fine, I guess I'll send another patch to make *ms local to<br>
find_mountpoint_check(), and rename some vars ("sb" and "mp" have their<br>
own expected meanings in xfsprogs)<br></blockquote></div><div class="gmail_extra"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​Hmmm, sorry about the mess... :-( I will look on the patch again too.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Speaking about  Coverity, is there a way I can get to it and use it myself? (Maybe it may be b​etter to continue this off list - from what I heard, Coverity guys don't like to be talk about publicly...)</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Cheers,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Jan</div><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Jan Tulak<br></div><a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a> / <a href="mailto:jan@tulak.me" target="_blank">jan@tulak.me</a></div></div></div></div>
</div></div>