getpass fgets check
vapier at gentoo.org
Sat Dec 31 23:13:16 UTC 2011
On Friday 23 December 2011 08:45:38 Joakim Tjernlund wrote:
> > From: Denys Vlasenko
> > On Tue, Dec 20, 2011 at 8:43 AM, Daniel Wainwright wrote:
> > > I believe there is a simple error in getpass.c, line 80:
> > >
> > > static char buf[PWD_BUFFER_SIZE];
> > I propose the following patch (sorry, it's attached, not inline).
> > * fixes bogus fgets error check
> > * fixes bogus strlen() < 0 check
> > * switches off buffering regardless of tcgetattr() success
> > * prints newline even on error or if there was no '\n' on input
> > * uses sizeof(buf) instead of PWD_BUFFER_SIZE
> +if (!fgets (buf, sizeof(buf)-1, in))
> Should probably be sizeof(buf) instead of sizeof(buf)-1 as per earlier
any response Denys ?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 836 bytes
Desc: This is a digitally signed message part.
More information about the uClibc