On Sun, 17 Feb 2013, Darren Tucker wrote:
> On Sun, Feb 17, 2013 at 01:46:29AM +1100, Darren Tucker wrote:
> > OK that's not intended behaviour. I'll look at it tomorrow and either fix
> > it or revert it.
>
> The problem was I didn't distinguish the default-provided IdentityFiles.
> Please try this.
>
> @@ -582,9 +602,7 @@ parse_yesnoask:
> if (*intptr >= SSH_MAX_IDENTITY_FILES)
> fatal("%.200s line %d: Too many identity files
> specified (max %d).",
> filename, linenum, SSH_MAX_IDENTITY_FILES);
> - charptr = &options->identity_files[*intptr];
> - *charptr = xstrdup(arg);
> - *intptr = *intptr + 1;
> + add_identity_file(options, NULL, arg, 1);
Does it make sense to remove the (*intptr >= SSH_MAX_IDENTITY_FILES)
check here too?
ok either way