On 20/02/2015 8:29 pm, Sebastian Huber wrote:

On 20/02/15 10:06, Nick Withers wrote:
On Fri, 2015-02-20 at 08:59 +0100, Sebastian Huber wrote:
>Hello Nick,
>
>thanks for the patch. I checked in a slightly different version which
>ignores the return status completely.
Hi Sebastian,

Your commit effectively reverts Joel's changes in
https://git.rtems.org/rtems/commit/?id=3211e8e9c0565fae8b13ba0115e3a23ad3ae8ade


Would chucking a (void) in front of the mkdir() shut Coverity up?

Ok, I should have checked the commit history since I wondered why I
added this error check in the first place. Lets try a (void) cast.


What about doing a stat, create the directory if not present and then error on a mkdir failure ? There are other failures that could happen making a directory that we should correctly report.

Chris
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to