[systemd-devel] [PATCH 2/2] tty-ask-password-agent: fix CID 996261

Greg KH gregkh at linuxfoundation.org
Mon Nov 17 08:56:09 PST 2014


On Mon, Nov 17, 2014 at 04:28:58PM +0530, Susant Sahani wrote:
> ---
>  src/tty-ask-password-agent/tty-ask-password-agent.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/tty-ask-password-agent/tty-ask-password-agent.c b/src/tty-ask-password-agent/tty-ask-password-agent.c
> index e6dc84b..1fc792b 100644
> --- a/src/tty-ask-password-agent/tty-ask-password-agent.c
> +++ b/src/tty-ask-password-agent/tty-ask-password-agent.c
> @@ -376,8 +376,8 @@ static int wall_tty_block(void) {
>                  return -ENOMEM;
>  
>          mkdir_parents_label(p, 0700);
> -        mkfifo(p, 0600);
>  
> +        (void)mkfifo(p, 0600);

You really aren't "fixing" anything in these patches, just merely
papering over the Coverity issues.  Which is fine, if you really want to
do that, but don't think it's anything other than that...

thanks,

greg k-h


More information about the systemd-devel mailing list