"Jacob S. Barrett" <[EMAIL PROTECTED]> wrote:
> Vendor IDs assigned in the dictionary appear to get sign extended in the 
> attributes stream.  FreeRADIUS allows only at 2 byte Vendor ID in the 
> dictionary, the RFC specifies 4 (first octet always 0).

  Yeah, that's a hack in the server, and should eventually be fixed.

>  When cast to a 4 byte value the sign is being carried which causes
> problems on the client when it decodes the 4 byte value as an
> unsigned 4 byte integer.

  The 'attribute' entry in the VALUE_PAIR structure should really be
'unsigned int', rather than 'int'.  A simpler short-term solution is
to fix the VENDOR macro in src/include/libradius.h, so that the number
it returns is only 16 bits long.

  I'll commit a fix today.

  Alan DeKok.

- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to