Patch: avpair.c - empty reply VALUE_PAIR (0x0) for replies with unknown vendor attributes

[Alan DeKok]

Adam Lewis wrote:
> I also seem to be having a problem with attributes that should be ignored but
> otherwise have no effect on the other attributes in a reply.
> 
> If an attribute from an unknown vendor happens to be the very last attribute
> in the list, rc_avpair_gen() ignores it by returning pair immediately. The
> only problem being pair is still set to the entry value of NULL and this
> ripples up through all the recursive calls, which means all the other
> attributes get ignored as well. Has anybody else seen this?

  No, but it seems to be a real issue.  I'm not sure why recursing over
the attributes is any easier than a simple loop.

  Alan DeKok.