Device Drivers Which Include mkdir("/dev")
joel at rtems.org
Thu Feb 18 17:20:29 UTC 2021
On Thu, Feb 18, 2021 at 11:06 AM Gedare Bloom <gedare at rtems.org> wrote:
> On Thu, Feb 18, 2021 at 8:56 AM Joel Sherrill <joel at rtems.org> wrote:
> > Hi
> > There are a lot of Coverity issues related to device drivers which call
> mkdir("/dev") and ignore the return value. My first thought was that they
> should have (void) added since /dev could have been created by an earlier
> > Then it occurred to me that libcsupport/src/base_fs.c includes
> mkdir("/dev") and a fatal error if it cannot create it.
> > Doesn't this mean that every call to mkdir("/dev") in a BSP or device
> driver is redundant? They should be removed since the base FS contents are
> always in place before any device drivers are called.
> > Thoughts?
> Seems reasonable. You could probably add an assert that /dev exists.
The creation of the device node will fail if /dev does not exist so I think
I think the call is pointless and changing it to stat() to make sure it
just adds bulk to the driver/BSP dependencies.
I think just deleting it seems prudent. There is an error path if it doesn't
> > --joel
> > _______________________________________________
> > devel mailing list
> > devel at rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the devel