http://defect.opensolaris.org/bz/show_bug.cgi?id=12222



--- Comment #6 from amaguire <alan.maguire at sun.com> 2009-10-27 16:42:41 UTC 
---
(In reply to comment #4)
> It looks like if you comment out 1837, the while loop doesn't actually do
> anything, and could all be replaced with just "(void) fclose(fp);"
> 

The indentation of the fclose(fp) on line 1837 is misleading;
the fclose() and subsequent setting of var_run_mounted are
all part of the same code block (note the opening parenthesis
on line 1836). If I had to guess, I'd say this might have
been the result of a mismerge at some point.

> It all just begs the question of what that code is really supposed to do.

As far as I can tell, the intent is to walk mnttab looking
for the /var/run tmpfs filesystem; if found, we set var_run_mounted
to 1.

-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.

Reply via email to