[krbdev.mit.edu #2170] Fixed problem with sig_t on krb5-1.3.1

[Ken Raeburn]

Michael Schloh von Bennewitz <krb5bugs at encambio.com> writes:
> It is very likely that this problem was local to OpenPKG, because of
> a patch applied that removes the sig_t typedef. I think you should
> assume that your code is correct, although there's a small chance
> that the unconditional typedef collides at buildtime with that of a
> FreeBSD header, for example. It seems that the sig_t to my_sig_t

Okay.  FreeBSD isn't one of our testing platforms at the moment,
unfortunately.

> detour does the trick, but I don't understand why there's no build
> error as my_sig_t is never defined:

The line

  typedef sigtype (*sig_t)();

after macro substitution will wind up looking something like this:

  typedef void (*my_sig_t)();

> The distribution we use is from:
>
>   http://www.crypto-publish.org/dist/mit-kerberos5/krb5-%{version}.tar.gz

Yep, that one should be good.  We don't control it, but we know the
guy who puts our stuff there.

> In closing, I conclude that this bug report was a false alarm. I
> hope you're not disappointed, but maybe what good came out of this
> is considering if autoconf could be a better test of sig_t presence.

I'll keep the ticket open for this, but it'll be lower priority since
the current code seems like it should work okay.

> Thanks for providing such a good piece of software as kerberos is,
> and keep up the good work.

Thanks for the kind words.

> P.S. I'm on a monthlong vacation as of tomorrow, so please don't be
> offended if I don't respond after then.

No problem.  Enjoy your vacation.

Ken