summaryrefslogtreecommitdiffstats
path: root/usr.sbin/syslogd
Commit message (Collapse)AuthorAgeFilesLines
* Also terminate program names if we hit a '/' - this is to be slightlydwmalone2004-05-301-1/+2
| | | | | | | more friendly to postfix log messages. PR: 50912 Submitted by: Stanislav Lapshansky <slapsh@slapsh.pp.ru>
* A log file name may now be prefixed by a '-' if it should not bedwmalone2004-05-302-2/+32
| | | | | | | | | | | | | explicitly fsynced after kernel messages are logged. This option should be syntax compatible with a similar option in Linux syslogd. I've made some small changes to Pekka's patch, hoepfully I haven't goofed anything. PR: 66790 Submitted by: Pekka Savola <pekkas@netcore.fi> Obtained from: Martin Schulze's syslogd MFC after: 1 month
* Try to be more careful about using using the file descriptor f_file.dwmalone2004-05-301-7/+13
| | | | | | | | | | | | | | | | | | | | | Syslogd should ensure that f_file is a valid file descriptor when f_type is FILE, CONSOLE, TTY and for a PIPE where f_pid > 0. If the descriptor is closed/invalid then the type should be set to UNUSED or the pid should be set to 0. To this end: 1) Don't close(f->f_file) if we can't send a message to a remote host because the file descriptor used for remote logging is stored in finet, not in f->f_file. f->f_file is probably uninitialised, so I guess we usually end up closing fd 0. 2) Don't close PIPE file descriptors if they are invalid. 3) If the call to p_open fails, don't set the pid. The OpenBSD patches in this area set f_file to -1 after the fd is closed and then avoids calling close if f_file < 0. I haven't done this, but it might be a good idea too. Inspired by: PR 67139/OpenBSD
* Some string fixes.dwmalone2004-05-291-4/+9
| | | | | | | | | | | 1) Use strncpy on strings out of utmp. 2) Avoid running off the start of one string while removing white space. (I've used slightly different code to OpenBSD here.) 3) Ignore trailing spaces in the priority. PR: 67139 Submitted by: Xin LI <delphij@FreeBSD.org.cn> Obtained from: OpenBSD
* Exit with a warning if the path to one of the logging sockets isdwmalone2004-05-291-0/+4
| | | | | | | too long, rather than silently truncating them. PR: 67139 Inspired by: OpenBSD
* Fix some bugs that don't manifest themselves in practice.dwmalone2004-05-291-3/+6
| | | | | | | | | | | | 1) Don't check for getopt returning '?', we have a default case. 2) Check if the priority is LOG_KERN correctly - in practice LOG_KERN is 0, so it makes no difference. OpenBSD fixed a different nearby bug that we don't have 'cos our definition of LOG_MAKEPRI is different to OpenBSD's. Copy a comment from OpenBSD, observing that LOG_KERN is 0. Inspired by PR: 67139
* Update a couple of comments.dwmalone2004-05-291-2/+2
| | | | | | PR: 67139 Submitted by: Xin LI <[3]delphij@FreeBSD.org.cn> Obtained from: OpenBSD
* Silence some constness and printf type warnings. Most of thedwmalone2004-05-261-12/+14
| | | | | const fixes are ugly 'cos the types in an iovec aren't quite right for a writev.
* Fixed style of DPADD and LDADD assignments as per style.Makefile(5).ru2004-02-051-2/+2
|
* Use strtol to parse the <%d> tags in printline() and printsys().iedowse2004-01-311-18/+18
| | | | | | Sort local variables declarations in these functions. Submitted by: bde
* Be much more strict about parsing tagged log messages from /dev/klog;iedowse2004-01-261-10/+20
| | | | | | | | | | | | if the line doesn't match ^<%d>, then treat it as a regular kernel printf line. Previously if a kernel printf message started with "<" it would be interpreted as a log message, often with LOG_EMERG level. This was triggered by some printfs in sys/dev/aic7xxx/, and can also happen with the partial lines that result if syslogd cannot keep up with the rate of arrival of kernel messages. Reviewed by: dwmalone MFC after: 1 week
* logerror is used in syslogd to log errors from syslogd itself. Itdwmalone2003-11-161-0/+6
| | | | | | | | | | | | is possible for an error to occur while trying to log an error, and this can result in infinite recursion (or at least until we run out of stack). Rather than this, we ignore requests to log an error while logging an error. PR: 51253 MFC after: 2 weeks
* Remove an argument to printf that is unused.dwmalone2003-11-161-1/+1
|
* mdoc(7): Use the new feature of the .In macro.ru2003-09-081-1/+1
|
* Avoid a corrupt timestamp on the consolegshapiro2003-05-171-2/+6
| | | | | | | PR: 51587 Submitted by: Dmitry Sivachenko <mitya@cavia.pp.ru> Approved by: re (rwatson) MFC after: 3 days
* Avoid code duplication by using libutil's trimdomain() instead ofgshapiro2003-05-172-10/+7
| | | | | | | | | an incorrect re-implementation. PR: 52223, 52342 Submitted by: Dan Nelson <dnelson@allantgroup.com> Approved by: re (bmah/rwatson) MFC after: 2 days
* Cleanup hostname matching in syslogd:gshapiro2003-05-041-17/+24
| | | | | | | | | | | | | | | | | | | 1. Hostnames were not treated case insensitively in all cases. 2. The method for stripping hostnames when reading the syslog.conf differed from that when finding the hostname of an incoming request. This lead to a broken match check. In my case, it meant I had to have '@scooter.smi.example.com.example.com' to have 'logger.example.com' properly save messages from 'scooter.smi.sendmail.com'. 3. Add paranoia to cfline() such that it doesn't try to access memory outside of the bounds of the f_host string. 4. While I am here, get rid of an outdated comment, argv[{0,1,2}] are now checked for NULL after the strdup() calls. Reviewed by: dwmalone MFC after: 1 week
* style.Makefile(5)obrien2003-04-041-1/+1
|
* Hostname specifications must allow commas in the value. They arethomas2003-02-211-1/+2
| | | | | | | used to separate multiple host names. Noted by: Dan Nelson <dnelson@allantgroup.com> Reviewed by: roberto
* Allow multiple hosts or programs to be named in programthomas2003-02-132-6/+39
| | | | | | | | | | | | | or host specifications, eg: !foo,bar *.* /var/log/only_foo_or_bar.log !-foo,bar *.* /var/log/all_except_foo_or_bar.log Reviewed by: roberto Not objected to by: arch@
* Various language tweaks.trhodes2002-12-241-17/+17
| | | | | PR: 45604 Submitted by: Chris Pepper <pepper@rockefeller.edu>
* Uniformly refer to a file system as "file system".ru2002-12-122-2/+2
| | | | Approved by: re
* mdoc(7) police: markup fixes.ru2002-11-272-4/+4
| | | | Approved by: re
* Factor out the code that determines whether a message must be skippedthomas2002-11-071-26/+27
| | | | | | | as a consequence of a host or program name specification into a common function, skip_pmessage. Reviewed by: roberto
* Add used #include <limits.h>.wollman2002-10-271-0/+1
|
* Sync usage() with reality and sort it alphabetically.tjr2002-10-181-4/+5
| | | | | | PR: 42620 Submitted by: Jeff Ito MFC after: 1 month
* Add -c option to synopsis. Sort it alphabetically, too.tjr2002-10-181-3/+3
|
* Avoid referencing a removed (and freed) queue entry. This partially revertsdes2002-10-061-2/+4
| | | | | | | | revision 1.101 (which did not introduce the bug but made it harder to fix) PR: misc/40363 Submitted by: David Dunham <dwdunham@isilon.com> MFC after: 2 weeks
* Introduce to the !program specification a capability similar to the onedcs2002-09-232-2/+25
| | | | | | | | | | | | | | | | | | | | | | that already exists for hosts: being able to specify a section that applies to every program *except* the one in question. The normal syntax for program specification is still valid. For the new capability, one uses: !-program Since there is no way to specify a program beginning with a dash in the old syntax, as it would be interpreted as the case above, the following alternative syntax to the original capability is provided: !+program This shouldn't introduce incompatibilities with any syslogd configuration in production because -stable's syslogd does not support a dash anywhere in the program specification. MFC after: 2 weeks
* Make syslogd -u treat "*" as all levels by explictly setting pri_cmpdwmalone2002-09-042-6/+24
| | | | | | | | | | | for it. While I'm here, add a the ability to say "!level" in a way which should be compatible with Linux's syslogd. PR: 28935 No objections: audit MFC after: 2 weeks
* Don't give up on a remote log host when we get a EHOSTUNREACH orcjc2002-08-251-3/+20
| | | | | | | | | | | | | | | | EHOSTDOWN. These are often transient errors (when the remote host reboots, temporary network problems, etc.), and we'd rather err on the side of caution and keep trying send messages that never arrive than just give up. Note that this is not an implementation of the "back-off" methods given in the PR. Those just seem too complicated. Why not just keep trying each time? Trying and failing doesn't really consume significantly more resources than if we were successful for each message. PR: bin/31029 MFC after: 1 week
* When parsing the program name from the incoming log messagerobert2002-07-251-1/+1
| | | | | | from a client, accept all printable characters as being part of the program name, except ':' and '[', because each is a possible delimiter.
* When reading a program name from the syslog configuration file,robert2002-07-251-1/+1
| | | | | | | | | | do not stop copying it into a buffer when encountering a non-alphanumerical character. Only stop at unprintable characters. This makes syslogd work correctly with executables like `interp.bin', `httpd_old', etc. PR: misc/40941 MFC after: 1 week
* use IPV6_V6ONLY instead of non standard IPV6_BINDV6ONLY.ume2002-07-221-3/+1
| | | | MFC after: 1 week
* The .Nm utilitycharnier2002-07-141-7/+7
|
* Consistancy check: s/file system/filesystem/gtrhodes2002-05-161-1/+1
|
* Remove spaces between (void) and function calls. Use TAILQ_FOREACHdes2002-03-241-30/+40
| | | | | instead of hand-rolled equivalent. Add curly braces for symmetry around else.
* Parenthesize return values and pull an orphaned { back into the fold.des2002-03-241-19/+18
|
* __FBSDID, ANSIfy function definitions, staticize functions and globals,des2002-03-241-148/+108
| | | | | don't initialize globals to 0, some reindentation of declarations, some comments added or reworded.
* Un-__P().des2002-03-241-27/+27
|
* Remove unnecessary setjmp.h.maxim2002-02-131-1/+0
| | | | | | | Reviewed by: ru Approved by: ru Obtained from: OpenBSD MFC after: 1 week
* Unlink all log sockets at startup.maxim2002-02-131-0/+1
| | | | | | | | PR: misc/34839 Reviewed by: ru Approved by: ru Obtained from: OpenBSD MFC after: 2 weeks
* mdoc(7) police: add missing markup bit.ru2002-01-101-1/+1
|
* Initialze a stack variable with NULL to unbreak buildworld with -Wall.deischen2002-01-091-1/+1
| | | | | I don't know about anyone else, but the compiler was always aborting on this stupid warning, and has been doing so for weeks.
* Add a new flag '-c' to disable repeated line compression when the outputarchie2002-01-052-2/+13
| | | | | | is a pipe to another program, or, if specified twice, in all cases. PR: bin/32420
* mdoc(7) police: markup nits.ru2001-12-081-1/+1
|
* Make the default kernel prefix "kernel:" instead of the boot file,dd2001-11-272-4/+25
| | | | | | | | | | | | | with the old behavior available via the -o option (it might still be useful if one has many kernels and cares which messages came from which). If the boot file is not used as the prefix, it is still logged once at startup. This change is prompted by the fact that the boot file is now much longer ("/boot/kernel/kernel" vs. "/kernel"), which significanlty bloats the syslogd output. Reviewed by: peter
* If the mask length is zero, there is no need to convert it to aps2001-11-141-1/+4
| | | | | | | | netmask. PR: 31947 Submitted by: Aaron Scarisbrick <aaronsca@hotmail.com> MFC after: 1 week
* Check the return values from calls to malloc(), calloc(), strdup() etc.des2001-09-301-12/+33
| | | | | Submitted by: Philippe Charnier <charnier@xp11.frmug.org> MFC after: 2 weeks
* Add a strcasecmp() call I forgot in my hurry to commit the previous fix.des2001-09-131-1/+2
| | | | | | | Without this call, the hostname check would in some cases yield false positives. Pointed out by: ru
OpenPOWER on IntegriCloud