from src/main/mainconfig.c ~line 402

---- paste ----
               /*
                 * Double check length, just to be sure!
                 */
                if (strlen(authhost) >= sizeof(c->server)) {
                        radlog(L_ERR, "%s[%d]: Server name of length %d is greater 
that allowed: %d",
                                        filename, cf_section_lineno(cs),
                                        strlen(authhost), sizeof(c->server) - 1);
                        return -1;
                }
                if (strlen(name2) >= sizeof(c->realm)) {
                        radlog(L_ERR, "%s[%d]: Realm name of length %d is greater than 
allowed %d",
                                        filename, cf_section_lineno(cs),
                                        strlen(name2), sizeof(c->server) - 1);
                        return -1;
                }
---------------

authhost is not checked for not being null before strlen, this leads to a segmentation 
fault
on FreeBSD 4.8 (atleast) when authhost is not in proxy.conf.

I wasted a few minutes as i had entered "authost" instead of "authhost" in the 
configuration file.

Not being a very strong C coder so im not too competent at diff/patch so i apologize.

This "feature" is also in the newest tarball

Just my 0.02NOK

// Sten

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

Reply via email to