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

Susant Sahani susant at redhat.com
Mon Nov 17 09:06:53 PST 2014


On 11/17/2014 10:26 PM, Greg KH wrote:
> 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...

Yes my intention is to for coverity only Any way next line 'open' 
handling the error case .

Susant


More information about the systemd-devel mailing list