#
347610 |
|
15-May-2019 |
ngie |
MFC r320009,r347075:
r320009 (by sbruno):
Quiesce clang warning while building lpc.
usr.sbin/lpr/lpc/lpc.c Warning passing 'char *[20]' to parameter of type 'const char **' discards qualifiers in nested pointer types [-Wincompatible-pointer-types-discards-qualifiers] Fix: Explicitly cast the variable "margv" to const char ** only for it's use as a parameter to suppress the error
r347075:
Fix `clang -Wcast-qual` issues
Remove unnecessary `char*` casting for arguments passed to `cget*(3)`, and deconst `_PATH_PRINTCAP` before passing it to `cget*` via the `printcapdb` variable.
This unblocks ^/projects/runtime-coverage-v2 from building cleanly on universe13a.freebsd.org. I suspect the issue was introduced through some changes to `bsd.*.mk` inclusion on the branch, which I will continue to investigate/isolate.
Tested with: clang 8 (arm64)
|
#
331722 |
|
29-Mar-2018 |
eadler |
Revert r330897:
This was intended to be a non-functional change. It wasn't. The commit message was thus wrong. In addition it broke arm, and merged crypto related code.
Revert with prejudice.
This revert skips files touched in r316370 since that commit was since MFCed. This revert also skips files that require $FreeBSD$ property changes.
Thank you to those who helped me get out of this mess including but not limited to gonzo, kevans, rgrimes.
Requested by: gjb (re)
|
#
330897 |
|
14-Mar-2018 |
eadler |
Partial merge of the SPDX changes
These changes are incomplete but are making it difficult to determine what other changes can/should be merged.
No objections from: pfg
|
#
302408 |
|
07-Jul-2016 |
gjb |
Copy head@r302406 to stable/11 as part of the 11.0-RELEASE cycle. Prune svn:mergeinfo from the new branch, as nothing has been merged here.
Additional commits post-branch will follow.
Approved by: re (implicit) Sponsored by: The FreeBSD Foundation |
#
298910 |
|
01-May-2016 |
araujo |
Use MIN macro from sys/param.h.
MFC after: 2 weeks.
|
#
297795 |
|
10-Apr-2016 |
pfg |
lpr: replace 0 with NULL for pointers.
Found with devel/coccinelle.
Reviewed by: gad
|
#
241852 |
|
22-Oct-2012 |
eadler |
Check the return error of set[ug]id. While this can never fail in the current version of FreeBSD, this isn't guarenteed by the API. Custom security modules, or future implementations of the setuid and setgid may fail.
PR: bin/172289 PR: bin/172290 PR: bin/172291 Submittud by: Erik Cederstrand <erik@cederstrand.dk> Discussed by: freebsd-security Approved by: cperciva MFC after: 1 week
|
#
230044 |
|
13-Jan-2012 |
kevlo |
fgets(3) returns a pointer, so compare against NULL, not integer 0.
|
#
216372 |
|
11-Dec-2010 |
joel |
Remove the advertising clause from UCB copyrighted files in usr.sbin. This is in accordance with the information provided at ftp://ftp.cs.berkeley.edu/pub/4bsd/README.Impt.License.Change
|
#
194494 |
|
19-Jun-2009 |
brooks |
In preparation for raising NGROUPS and NGROUPS_MAX, change base system callers of getgroups(), getgrouplist(), and setgroups() to allocate buffers dynamically. Specifically, allocate a buffer of size sysconf(_SC_NGROUPS_MAX)+1 (+2 in a few cases to allow for overflow).
This (or similar gymnastics) is required for the code to actually follow the POSIX.1-2008 specification where {NGROUPS_MAX} may differ at runtime and where getgroups may return {NGROUPS_MAX}+1 results on systems like FreeBSD which include the primary group.
In id(1), don't pointlessly add the primary group to the list of all groups, it is always the first result from getgroups(). In principle the old code was more portable, but this was only done in one of the two places where getgroups() was called to the overall effect was pointless.
Document the actual POSIX requirements in the getgroups(2) and setgroups(2) manpages. We do not yet support a dynamic NGROUPS, but we may in the future.
MFC after: 2 weeks
|
#
151476 |
|
19-Oct-2005 |
stefanf |
Use the new name H_SETSIZE instead of the old H_EVENT to set the history size.
PR: 86355 Approved by: gad
|
#
142199 |
|
22-Feb-2005 |
delphij |
MFS5: Minor style(9) tweak.
|
#
141947 |
|
15-Feb-2005 |
delphij |
Be more careful when doing el_parse() - only do it when el is properly initialized, that happens when lpc is called from a tty. Without this change, it's possible to get SIGSEGV simply doing: echo "..:" | lpc
Reported by: Wojciech A. Koszek <dunstan at freebsd czest pl> PR: 77462 (patch rewritten by myself) MFC After: 1 week
|
#
121065 |
|
13-Oct-2003 |
tjr |
Fix two buffer overflows caused by off-by-one errors: avoid writing a null character 1 byte past the end of cmdline[] when libedit is being used for input, and avoid writing a null pointer 1 element past the end of margv[].
Reviewed by: gad
|
#
117599 |
|
14-Jul-2003 |
gad |
More changes to use __FBSDID() for setting rcsids, and fix the format of 'sccsid' lines so they consistently match style(9) guidelines. Note that this means you will have to add '-a' to the 'strings' command when searching for rcs ids, eg: strings -a /usr/sbin/lpc | grep '$FreeBSD'
Reviewed by: discussed on cvs-src & with bde and obrien MFC after: 15 days
|
#
100203 |
|
16-Jul-2002 |
gad |
Changes which rewrite 'lpc topq', and which add 'lpc bottomq'. These reflect much valuable feedback from wollman. More details on the new 'lpc topq' are in the log message for revision 1.2 of lpc/movejobs.c.
The previous implementation of 'lpc topq' is available as 'lpc xtopq', in case there are any problems noticed in the new implementation. If there are no problems with this version, a later update will remove the 'lpc xtopq' command.
Reviewed by: freebsd-print@bostonradio.org MFC after: 6 days
|
#
98267 |
|
15-Jun-2002 |
gad |
Add a new command to 'lpc' called 'setstatus', which would be used to change the status message of a print queue. This includes some minor changes to the upstat() routine, so that error messages are not printed while seteuid(priv-user).
Reviewed by: freebsd-audit and freebsd-print@bostonradio.org MFC after: 10 days
|
#
84262 |
|
01-Oct-2001 |
obrien |
Adjust to the libedit enhancements some functions now take more parameters.
Partially submitted by: kris
|
#
84261 |
|
01-Oct-2001 |
obrien |
*** empty log message ***
|
#
83563 |
|
17-Sep-2001 |
gad |
Fix so that lpc's interactive-mode will not be confused by EditLine processing into thinking that there is a print-queue called 'xterm'...
Reviewed by: short discussion on freebsd-stable MFC after: 1 week
|
#
80173 |
|
22-Jul-2001 |
gad |
Get rid of a compile-time warning by casting to (size_t).
MFC after: 1 week
|
#
79742 |
|
15-Jul-2001 |
gad |
Get rid of one compile-time warning by changing an 'int' to a 'size_t'.
MFC after: 1 week
|
#
78750 |
|
25-Jun-2001 |
gad |
Make 'lpc clean' somewhat safer. Add an 'lpc tclean' command, which allows one to see what files would be removed *if* an 'lpc clean' is done. 'tclean' will remove no files, and is therefore not a privileged command. Also, both 'lpc clean' and 'lpc tclean' will now look for 'core' files in spool directories (but not remove them). They also print out an extra line of info when a datafile to be removed is a symlink (from 'lpr -s'), saying what file it is a symlink to.
The 'lpc clean' commands also now print out a summary line saying how many queues were checked, how many files were removed (or "would be" removed, for tclean), and how much disk space is involved. For the benefit of those who have many print queues, 'lpc clean all' will only print out the names of print queues where some "interesting" files were found, instead of printing out a header-line for every queue in your printcap file.
Reviewed by: freebsd-print@bostonradio.org freebsd-audit@FreeBSD.org MFC after: 2 weeks
|
#
78280 |
|
15-Jun-2001 |
gad |
Rename global variable 'name' to 'progname', thus fixing a number of warnings which come up for various routines that have a parameter which is also called 'name'.
Reviewed by: freebsd-print@bostonradio.org MFC after: 1 week
|
#
78146 |
|
12-Jun-2001 |
gad |
Fix about 90-100 warnings one gets when trying to compile lpr&friends with BDECFLAGS on, mainly by adding 'const' to parameters in a number of routine declarations. While I'm at it, ANSI-fy all of the routine declarations. The resulting object code is exactly the same after this update as before it, with the exception of one unavoidable change to lpd.o on freebsd/alpha.
Also added $FreeBSD$ line to lpc/extern.h lpc/lpc.h lptest/lptest.c
Reviewed by: /sbin/md5, and no feedback from freebsd-audit
|
#
68400 |
|
06-Nov-2000 |
gad |
Get rid of a minor compile-time warning.
|
#
65035 |
|
23-Aug-2000 |
alfred |
the code assumes that getgroups() always returns NGROUPS groups, however that is not true. Instead of looping NGROUPS times, get the return value from getgroups() and loop over the return that many times.
Noticed by: David A. Holland <dholland@eecs.harvard.edu>
|
#
62294 |
|
30-Jun-2000 |
mph |
Quit on EOF from terminal instead of redisplaying the prompt.
|
#
50479 |
|
27-Aug-1999 |
peter |
$Id$ -> $FreeBSD$
|
#
50077 |
|
20-Aug-1999 |
mdodd |
Fix some cut and paste damage.
Noticed by: Norman C. Rice <nrice@emu.sourcee.com>
|
#
50071 |
|
19-Aug-1999 |
mdodd |
What the heck was I thinking? Nobody else saw this? Sheesh.
(num > MAX) ? MAX : num
rather than
(MAX > num) ? MAX : num
Also, make things a little easier to read while I'm here.
|
#
50042 |
|
19-Aug-1999 |
mdodd |
Use el_source() so we pick up .editrc
|
#
50039 |
|
19-Aug-1999 |
mdodd |
Add support for command line editing and history.
Remove src/contrib/bind/bin/nslookup/commands.c as it is generated by lex from commands.l.
Submitted by: lpc/cdcontrol patches originally by msmith. Reviewed by: msmith (in theory)
|
#
39084 |
|
11-Sep-1998 |
wollman |
Fix additional warnings. Remove -Werror, since some people have complained about it.
PR: 7886 Submitted by: Stefan Eggers <seggers@semyam.dinoco.de> (partially)
|
#
34784 |
|
22-Mar-1998 |
jb |
Cast an argument to int for a printf field width the way that gcc prefers it. This source is compiled with -Werror so the slightest warning is enough to ruin my day.
|
#
31492 |
|
02-Dec-1997 |
wollman |
Mega lpd/lpd upgrade, part I:
- Get rid of a lot of the static variables which were shared by many routines and programs in the suite. - Create an abstract interface to the printcap database, so that other retrieval and iteration mechanisms could be developed (e.g., YP, Hesiod, or automatic retrieval from a trusted server). - Give each capability a human-readable name in addition to the historic two-character one. - Otherwise generally clean up a lot of dark corners. Many still remain. - When submitting jobs, use the official login name record (from getlogin()) if there is one, rather than reverse-mapping the uid.
More to come...
|
#
29780 |
|
24-Sep-1997 |
charnier |
Use err(3). Add usage(). Various fixes in man pages.
|
#
27618 |
|
22-Jul-1997 |
imp |
Use setuid/seteuid around dangerous operations. Also a few buffer overflow patches that were "near" to where these operations are taking place. The buffer overflows are from OpenBSD. The setuid/seteuid patches are from NetBSD by way of OpenBSD (they changed them a little), at least from my read of the tree.
This is the first of a series of OpenBSD lpr/et al merges. It (and them) should be merged back into 2.2 and/or 2.1 (if requested) branches when they have been shaken out in -current. Obtained from: OpenBSD
|
#
19202 |
|
27-Oct-1996 |
imp |
lpc/cmds.c: From NetBSD via OpenBSD to fix NetBSD PR #506 More descriptive message for printer status (OpenBSD: 1.2)
Various warnings cleaned up (OpenBSD: 1.4)
lpc/lpc.c: Various warnings cleaned up (OpenBSD: 1.3)
lpd/lpd.c: Remove trailing blank lines (OpenBSD: 1.2)
Potential umask problem with creating /dev/printer (OpenBSD: 1.4 and 1.5)
Ftp bounce attack (untested on FreeBSD) (OpenBSD: 1.6, 1.8, 1.9) Fencepost in strncpy (OpenBSD: 1.6)
lpd/printjob.c: Fix from freebsd for waiting for an exiting filter, that appears not in the FreeBSD CVS tree. (OpenBSD: 1.6)
lpd/recvjob.c: Buffer overflow protection: use strncpy rather than strcpy. (OpenBSD: 1.3)
lpr/lpr.c: NetBSD change of return type for main() (OpenBSD: 1.2)
Restrict time running as root (OpenBSD: 1.7)
Use getcwd rather than getwd (from NetBSD)
Use snprintf rather than sprintf (OpenBSD: 1.8)
Minor tweak to end of loop and buffer overflow sanity. card() overflow already in FreeBSD (OpenBSD: 1.9)
lptest/lptest.c: void -> int return type of main, from NetBSD via OpenBSD (OpenBSD: 1.2)
pac/pac.c: void -> int return type of main, from NetBSD via OpenBSD (OpenBSD: 1.3)
Obtained from: OpenBSD
|
#
15703 |
|
09-May-1996 |
joerg |
Cleanup.
The removed files are no longer needed, they are actually labelled as ``Use only if you are not 4.4BSD''. (Yeah, the ol' crufty printcap.c is really gone!)
Properly declare all external objects in files ending in .h, as opposed to embed them into files ending in .c.
|
#
15638 |
|
05-May-1996 |
joerg |
This commit was generated by cvs2svn to compensate for changes in r15637, which included commits to RCS files with non-trunk default branches.
|
#
15637 |
|
05-May-1996 |
joerg |
Vendor-branch import of the 4.4BSD-Lite2 code for lpr. There are several bugfixes in it that are worth considering.
Don't be alarmed about the import conflicts...
Obtained from: 4.4BSD-Lite2
|
#
1553 |
|
26-May-1994 |
rgrimes |
BSD 4.4 Lite usr.sbin Sources
|