[PATCH] cpulimit: Explicitly ignore pa_read() return value.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2012-04-03 at 10:18 +0300, Tanu Kaskinen wrote:
> It doesn't matter if the function fails (I'm not sure if
> it's even possible), because the read data isn't used for
> anything and the daemon will terminate in any case. The
> void cast should get rid of a Coverity warning.
> 
> Removing the whole pa_read() call should be ok too, but I
> guess it's nice to clean up the pipe before terminating...
> ---
>  src/daemon/cpulimit.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/daemon/cpulimit.c b/src/daemon/cpulimit.c
> index 3a97297..0abbac0 100644
> --- a/src/daemon/cpulimit.c
> +++ b/src/daemon/cpulimit.c
> @@ -167,7 +167,7 @@ static void callback(pa_mainloop_api*m, pa_io_event*e, int fd, pa_io_event_flags
>  
>      pa_log("Received request to terminate due to CPU overload.");
>  
> -    pa_read(the_pipe[0], &c, sizeof(c), NULL);
> +    (void) pa_read(the_pipe[0], &c, sizeof(c), NULL);
>      m->quit(m, 1); /* Quit the main loop */
>  }
>  

This is now applied.

-- 
Tanu



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux