Martin Schulze wrote: > --- mod_auth_radius.c~ 2003-03-24 20:16:15.000000000 +0100 > +++ mod_auth_radius.c 2005-01-13 13:01:42.000000000 +0100 > @@ -971,8 +971,11 @@ find_attribute(radius_packet_t *packet, > } > return attr; > } > -#define radcpy(STRING, ATTR) {memcpy(STRING, ATTR->data, ATTR->length - 2); \ > - (STRING)[ATTR->length - 2] = 0;} > +#define radcpy(STRING, ATTR) do { \ > + unsigned char len = ATTR->length; \ > + if (len >= 2) len-=2; \ > + memcpy(STRING, ATTR->data, len); \ > + (STRING)[len] = 0;} until (0) ^^^^^
Make that "while" and it'll work. Too much pascal reminiscense in my head or something... I'm so sorry about the confusion I have created. Regards, Joey -- Ten years and still binary compatible. -- XFree86 Please always Cc to me when replying to me on the lists. -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]