[PATCH] pinentry-curses: Make sure we've got a real terminal

Werner Koch wk at gnupg.org
Fri Oct 24 21:01:01 CEST 2014


On Tue, 14 Oct 2014 14:10, mk at mkoskar.com said:

> I think more robust solution is for pinentry to make sure it "talks" to real
> terminal device, hence following patch. It is solving problem above, having
> pinentry to fail right away as expected.

Any other opinons on this?

> +      if (!isatty(fileno(ttyfi)) || !isatty(fileno(ttyfo)))
> +        return -1;
>        screen = newterm (tty_type, ttyfo, ttyfi);

Hmmm, we don't check for an error from newterm?  At least for memory
problems it will return NULL.  That code is pretty old , I can't
remember the details.


Salam-Shalom,

   Werner


-- 
Die Gedanken sind frei.  Ausnahmen regelt ein Bundesgesetz.




More information about the Gnupg-devel mailing list