Hello Adam,<br><br>I guess my description was a bit confusing - "first recursion" means the first recursion instance in that bit of messy code ;-). So wherever an unknown vendor happens the reply pointer is NULL. <br>
<br>Try my patch and see if it helps. If I remember correctly someone on the list wanted to submit another patch that translates unknown attributes/vendors into a special syntax - but I'm not sure if it has already been submitted.<br>
<br>Rgds,<br>Stefan<br><br><div class="gmail_quote">On Tue, Jun 15, 2010 at 10:12 AM, Adam Lewis <span dir="ltr"><<a href="mailto:VNQM87@motorola.com">VNQM87@motorola.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
I also seem to be having a problem with attributes that should be ignored but<br>
otherwise have no effect on the other attributes in a reply.<br>
<br>
If an attribute from an unknown vendor happens to be the very last attribute<br>
in the list, rc_avpair_gen() ignores it by returning pair immediately. The<br>
only problem being pair is still set to the entry value of NULL and this<br>
ripples up through all the recursive calls, which means all the other<br>
attributes get ignored as well. Has anybody else seen this?<br>
<br>
Adam.<br>
<div><div></div><div class="h5"><br>
<br>
Stefan Karss wrote:<br>
><br>
> Hello again,<br>
><br>
> when the RADIUS servers sends a reply packet with an attribute for an<br>
> unknown vendor freeradius-client just skips over this when filling<br>
> response<br>
> VALUE_PAIR list. Unfortunately it doesn't for unknown attributes from<br>
> known<br>
> vendors - in that case rc_avpair_gen returns NULL in the first recursion<br>
> which results in a NULL pointer for reply VALUE_PAIR.<br>
><br>
> The attached patch just skips over unknown attributes for known vendors<br>
> just<br>
> like the lib skips over attributes from unknown vendors.<br>
><br>
> Rgds,<br>
> Stefan<br>
><br>
><br>
</div></div><div class="im">> -<br>
> List info/subscribe/unsubscribe? See<br>
> <a href="http://www.freeradius.org/list/devel.html" target="_blank">http://www.freeradius.org/list/devel.html</a><br>
><br>
<br>
</div><font color="#888888">--<br>
View this message in context: <a href="http://old.nabble.com/Patch%3A-avpair.c---empty-reply-VALUE_PAIR-%280x0%29-for-replies-with--unknown-vendor-attributes-tp28313256p28888431.html" target="_blank">http://old.nabble.com/Patch%3A-avpair.c---empty-reply-VALUE_PAIR-%280x0%29-for-replies-with--unknown-vendor-attributes-tp28313256p28888431.html</a><br>

Sent from the FreeRadius - Dev mailing list archive at Nabble.com.<br>
</font><div><div></div><div class="h5"><br>
-<br>
List info/subscribe/unsubscribe? See <a href="http://www.freeradius.org/list/devel.html" target="_blank">http://www.freeradius.org/list/devel.html</a><br>
</div></div></blockquote></div><br>