summaryrefslogtreecommitdiff
path: root/usr.sbin/radiusd
AgeCommit message (Collapse)Author
2018-08-01Fix segmentation fault on radiusd(8) when exiting.Ricardo Mestre
If one of the configured modules doesn't have a secret setup then module->secret == NULL which would call strlen(NULL), within freezero(3), and that shouldn't happen, but in this case since the call is done it segfaults and the daemon is not properly shutdown. cluebat stick provided by semarie@, OK tb@ and deraadt@
2018-07-09No need to mention which memory allocation entry point failed (malloc,Kenneth R Westerback
calloc or strdup), we just need to log that we ran out of memory in a particular function. Recommended by florian@ and deraadt@ ok benno@ henning@ tb@
2018-07-08Be consistent in warn() and log_warn() usage whenKenneth R Westerback
running out of memory. Next step, be correct *and* consistent. ok dennis@ tb@ benno@ schwarze@
2017-10-17add missing HISTORY; based on CVS logs and release announcementsIngo Schwarze
2017-08-21Use waitpid()/EINTR idiom for the specific pid, rather than generic wait(),Theo de Raadt
in case the parent process was started with a dangling child. This style ensures any potential parent:child interlock isn't disrupted due to the "wrong" child being waited on first. Then the other other childs can safely zombie. ok millert jca brynet
2017-06-13Initialize the length parameter for radius_get_vs_raw_attr() sinceYASUOKA Masahiko
it's read/write. diff from IIJ.
2017-05-30Don't call TAILQ_REMOVE twice. This caused radiusd crash if it has aYASUOKA Masahiko
pending request when it stops.
2017-05-21A few more freezero() usesTheo de Raadt
ok yasuoka mikeb
2017-03-29Bring radiusd log.c copyright in line with other program's log.cAlexander Bluhm
and other radiusd source files. Remove the LOSS OF MIND clause. OK henning@ yasuoka@ deraadt@
2017-03-13Fix typo in FILES sectionJeremie Courreges-Anglas
from Pierre Emeriaud
2017-02-21Stop using "DEBUG" make variable for extra debugs.YASUOKA Masahiko
2016-09-20some fixes from rob pierce; ok yasuokaJason McIntyre
2016-08-27Add missing $OpenBSD$ line and remove an unnecessary comment line.YASUOKA Masahiko
2016-04-16Change last non-/gnu/ fcntl(x, F_GETFL, 0) strays to fcntl(x, F_GETFL).Kenneth R Westerback
No functional change. ok millert@
2016-04-13remove "abort" promise from debugging code in radiusdSebastien Marie
it is the default now, and the promise name isn't valid anymore. ok yasuoka@
2016-04-05Move more fcntl(,F_GETFL,0) -> fcntl(,F_GETFL).Kenneth R Westerback
No functional change. ok guenther@
2016-03-21Instead of creating a socket with socket() or accept() and thenPhilip Guenther
setting the O_NONBLOCK flag on it with fcntl(F_SETFL) afterwards, just pass SOCK_NONBLOCK to socket() or accept4() and get it right to begin with. ok millert@ krw@ beck@ deraadt@ jca@
2016-02-09fix a use after free in an error pathJonathan Gray
ok yasuoka@
2015-12-31Do not mix EX_* from sysexits.h and EXIT_* from stdlib.h, just useTodd C. Miller
EXIT_*.
2015-12-05EAGAIN handling for imsg_read. OK henning@ benno@Claudio Jeker
2015-12-05Remove NULL-checks before free()mmcc
2015-12-01Fix a couple typos. ok yasuoka@mmcc
2015-11-03Blank lines around pledge call.mmcc
2015-10-27Free the received radius packet when it is duplicated.YASUOKA Masahiko
diff from Yuuichi Someya
2015-10-27Initialize module_radius_req before use it.YASUOKA Masahiko
diff from Yuuichi Someya
2015-10-27Set O_NONBLOCK for UDP sockets not to block on recv(). ActuallyYASUOKA Masahiko
block had happened if an error of the socket is handled by send(). diff from Yuuichi Someya.
2015-10-27Fix radiusd module to set O_NONBLOCK properly.YASUOKA Masahiko
diff from Yuuichi Someya.
2015-10-26Using realm for bsdauth is misleading. It is currently unusable.YASUOKA Masahiko
2015-10-19Add pledge(2) for radiusctl(8) and radiusd(8).YASUOKA Masahiko
- radiusd: "stdio inet" - radiusd_radius: "stdio inet" - radiusd_bsdauth: - "stdio proc" for the non-priviledged process - "stdio getpw rpath proc exec" for the priviledged process - radiusctl: "stdio dns inet" "go ahead" deraadt
2015-10-19Remove a duplicated '#include <stdio.h>' line.YASUOKA Masahiko
2015-10-19Avoid a NULL dereference when getgrnam_r() returns NULL for `result'.YASUOKA Masahiko
2015-10-19Call tzset() before dropping the priviledge to use correct timezone.YASUOKA Masahiko
2015-10-19Can't assert "module->fd >= 0" in radiusd_stop() since the module mayYASUOKA Masahiko
be closed already when error.
2015-09-22typo.Igor Sobrado
2015-08-27Add a boundary check for safety and use snprintf() to construct the IPYASUOKA Masahiko
address strings instead of strl{cpy,cat}(). Also make the function return a valid string even in failure case. input deraadt ok deraadt
2015-08-26fix a use after free in an error path found with aflJonathan Gray
ok yasuoka@
2015-08-25Remove -h command line option from radiusd(8) to make it better style.YASUOKA Masahiko
Patch from Michael Reed
2015-08-21spacesTheo de Raadt
2015-08-03fix up previous;Jason McIntyre
2015-08-03Make radiusd.conf.5 describe all configuration optionsYASUOKA Masahiko
2015-08-02Check the received packet length properly.YASUOKA Masahiko
2015-08-02Fire pending events when the module starts.YASUOKA Masahiko
2015-08-02Tweak XXX comments.YASUOKA Masahiko
2015-08-02radiusd_module with libevent didn't stop when the daemon stops. FixYASUOKA Masahiko
it to delete its event handler properly.
2015-08-02Fix radiusd_radius to use syslog(3). It had used stderr for debug.YASUOKA Masahiko
2015-08-02Fix radiusd to start without -d. Also stop using event_initialized()YASUOKA Masahiko
to check whether the event handler is set.
2015-07-30Use the dedicated log function instead of log_warn(). Also, fix theYASUOKA Masahiko
variable name again.
2015-07-30Fix the variable name for struct radius_server. It was confused.YASUOKA Masahiko
2015-07-30Fix typo in comment.YASUOKA Masahiko
2015-07-30q_id didn't pass to radiusd_radius properly. It should be u_int not u_int.YASUOKA Masahiko