Rather than comparing the two keys' `id' slots, just compare the
pointers. It's not like we got them from different keyrings or
anything. This makes it clear that we're checking that we really have
the /same/ key, found by two different paths.
Fortunately, this isn't a security problem: the code in `key/key-io.c'
won't allow two keys with the same `id' value to exist in the same
keyring. This change therefore doesn't have any observable effect.
But, seeing as I just had a panic regarding a potential security hole in
`tripe-keys', it's as well to make it clear what's actually going on.
(unsigned long)s.keyid);
exit(EXIT_FAILURE);
}
(unsigned long)s.keyid);
exit(EXIT_FAILURE);
}
- if (kk && k->id != kk->id) {
if (v.verb) {
dstr_reset(&d); key_fulltag(k, &d);
dstr_reset(&dd); key_fulltag(kk, &dd);
if (v.verb) {
dstr_reset(&d); key_fulltag(k, &d);
dstr_reset(&dd); key_fulltag(kk, &dd);