MFC r317596:
mixer(8): Prevent possible sscanf() overflow.

Fix %s buffer sizes in sscanf().

Obtained from: DragonflyBSD (git dab952e2)

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

mixer(8): Style: Tag no-return usage() as __dead2

Coverity really should have figured this out from the exit(3) call at the end
of the routine, but just make it explicit.

No functional change.

Reported by: Coverity
CID: 1304866 (false positive double-close of 'baz')
Sponsored by: EMC / Isilon Storage Division

Return proper error message if recording device is not specified.

MFC after: 1 week

Mark global functions and/or variables in mixer(8) static where possible.

This allows compilers and static analyzers to do more thorough analysis.

Fix a typo.

Clean up code and fix style(9) nits.

- Check device masks and bypass printing headers if capable device does
not exist. /etc/rc.d/mixer tried to restore non-existent recording
source because /var/var/db/mixer*-state contains '=rec' for example.
- Remove hardcoded mixer2 and mixer3 and do the right thing.
- Replace getopt(3) with hand-rolled option parser. It was not able
to handle 'mixer -rec mic' although it is a valid command.
- Make '-s' and '-S' mutualy exclusive as mixer(8) says.
- Do not re-read recording source unless it has been modified.

- Fix possible memory leak due to repetetive strdup(3) without matching
free(3).
- print_recsrc() should honor shortflag and give appropriate output for
later consumption by /etc/rc.d/mixer. This will ensure that recording
device selections survived across reboot. Output everything to stdout
instead if stderr.

MFC after: 3 days

Add the -S flag which produces the -s output without
mixing field separators.

Userland signed char fixes for PPC build. Problems were using a char
return for getopt() and comparing to -1, ditto with fgetc() and EOF,
and using the kg_nice value from <sys/user.h>

Submitted by: Stefan Farfeleder <stefan@fafoe.narf.at>
Reviewed by: obrien, bde (a while back)
Tested lightly on: ppc, i386, make universe

Print out the current recording devices in the default printout.

Discussed with: cg

Break some particularly long lines.

Still not enough for: bde.

Use __FBSDID over rcsid[]. Protect copyright[] where needed.

Implement relative mixer adjustment.

PR: 46679 (similar functionality)

Print old mixer levels when setting new ones.

Constify and set WARNS=2.

Submitted by: Mike Barcroft <mike@q9media.com
Reviewed by: md5(1)

Change the behaviour of mixer slightly such that not explicitly specifying
a mixer device will set/get the master volume levels

PR: 25156
Submitted by: Tony Finch <dot@dotat.at>
Reviewed by: cg, nik

Fix format string warnings

Several cleanups:
- use getopt(3) to parse command line arguments instead of home-made incomplete
parser;
- be more verbose when error in command line encountered (i.e. incorrect
playing/recording device, incorrect recsrc syntax).

I've always wanted to be able to dump out the current values of
the mixer in a format suitable for saving and handing off to
mixer(1) at some future date to restore the settings.
Add a -s flag to do this.

remove #ifdef FreeBSD -> <machine/soundcard.h> special case as we have it
in the standard location now.

$Id$ -> $FreeBSD$

fix a core dump problem when you don't give -f a parameter... also compare
results of strcmp against 0 instead of !'ing results...

Closes PR: 6538

According to Bruce, err(-1,..) is bogus. Change it to err(1,..). More commits
to come in this area.

Use err(3).

Compact usage string, devices list can be too long

major modifications to the mixer. currently it doesn't report errors
that it encounters (such as invalid device), but does print usage upon
this error...

but otherwise allows complex commands such as:
mixer -f /dev/mixer4 synth line 50 -rec mic +rec line =rec cd ^rec mix

Make mixer print out all of the current mixer settings by defualt
instead of making the user run a separate command for each
setting they are interested in. Closes PR#432.

Fixed to not core dump if not enough arguments are given
with the +rec and -rec options.

Remove trailing whitespace.

This commit was generated by cvs2svn to compensate for changes in r6448,
which included commits to RCS files with non-trunk default branches.

non-X mixer, useful into rc.local