[PATCH 19/20] libcsupport/src/sync.c: Explicitly ignore return status

Sebastian Huber sebastian.huber at embedded-brains.de
Wed Nov 26 07:05:28 UTC 2014


On 26/11/14 00:02, Joel Sherrill wrote:
> From: Josh Oguin <josh.oguin at oarcorp.com>
>
> CodeSonar spotted that the return values were being ignored.
> ---
>   cpukit/libcsupport/src/sync.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/cpukit/libcsupport/src/sync.c b/cpukit/libcsupport/src/sync.c
> index b8e5059..0e935ce 100644
> --- a/cpukit/libcsupport/src/sync.c
> +++ b/cpukit/libcsupport/src/sync.c
> @@ -43,8 +43,10 @@ static void sync_wrapper(FILE *f)
>      *  matter if they succeed.  We are just making a best faith attempt
>      *  at both and trusting that we were passed a good FILE pointer.
>      */
> -  (void) fsync(fn);
> -  (void) fdatasync(fn);
> +  if ( fn != -1 ) {
> +    (void) fsync(fn);
> +    (void) fdatasync(fn);
> +  }
>   }
>   
>   /* iterate over all FILE *'s for this thread */

This test is superfluous since both functions validate the file 
descriptor.  All other patches look good.

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.




More information about the devel mailing list