[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