I think I have discovered a bug in the 20000823 snapshot of session.c ... Beginning at line 1229: if (screen != NULL) fprintf(stderr, "Adding %.*s/unix%s %s %s\n", screen-display, display, screen, auth_proto, auth_data); and repeated at line 1241: if (screen != NULL) fprintf(f, "add %.*s/unix%s %s %s\n", screen-display, display, screen, auth_proto, auth_data); Both fprintf() calls have four string arguments, but contain five variables. What do you think?
On Thu, 24 Aug 2000, Mark Miller wrote:> I think I have discovered a bug in the 20000823 snapshot of session.c ... > > Beginning at line 1229: > > if (screen != NULL) > fprintf(stderr, > "Adding %.*s/unix%s %s %s\n", > Both fprintf() calls have four string arguments, but contain five variables. > > What do you think?I think that you should read the manpage for printf :) the .* construct specifies the width of the format using one of the arguments. -d -- | "Bombay is 250ms from New York in the new world order" - Alan Cox | Damien Miller - http://www.mindrot.org/ | Email: djm at mindrot.org (home) -or- djm at ibs.com.au (work)
> > I think that you should read the manpage for printf :)Doh! Yep, so much for jumping the gun. :)> the .* construct specifies the width of the format using one of the > arguments.Indeed, something that I should have looked into before opening my big yap! Thanks.