Ticket History #3911: getaddrinfo code uses vars outside of storage duration

Revision 18181 reverts the change to plugins.c, which was checked in
unintentionally.

Also, I think this change was a little too aggressive in pushing
variables around to outer scope when they didn't need to be. Working on
a revision.

Revert part of previous changes: Move temporary values that don't need
longer storage duration, like error codes, back into GET_*_BY_* macros.

Commit By: raeburn



Revision: 18182
Changed Files:
U trunk/src/util/support/fake-addrinfo.c

pull up r18180:18182 from trunk

r18180@cathode-dark-space: raeburn | 2006-06-20 19:59:11 -0400
ticket: new
subject: getaddrinfo code uses vars outside of storage duration

The calls to gethostbyname_r and friends were wrapped in macros that declared
automatic variables for the auxiliary storage needed, but the pointers returned
by the functions would be used outside of that scope, when the storage would no
longer be valid.

Changed the macro interfaces to define new types for the auxiliary storage, and
add a new argument to the macros referring to that storage. Changed call sites
accordingly.


r18181@cathode-dark-space: raeburn | 2006-06-20 19:59:56 -0400
ticket: 18180

Revert an untested change that wasn't supposed to go into this ticket.

r18182@cathode-dark-space: raeburn | 2006-06-20 20:12:03 -0400
ticket: 3911

Revert part of previous changes: Move temporary values that don't need
longer storage duration, like error codes, back into GET_*_BY_* macros.


Commit By: tlyu



Revision: 18225
Changed Files:
U branches/krb5-1-5/src/util/support/fake-addrinfo.c