Avoid overflows when reading strings.

elide recently introduced warnings that trigger in normal cases, eg:

savecore: dump_exists:471: kvm_read newdumpmag: _kvm_kvatop(ffffffff8180030c)

avoid infinite loop in kmem_check(). fixes bug introduced in previous.

put a bunch of the kvm_read + warn on failure code into a macro that
describes more about what failed. now errors tell you which actual
variable was being requested instead of simply saying "not you

put a bunch of the kvm_read + warn on failure code into a macro that
describes more about what failed. now errors tell you which actual
variable was being requested instead of simply saying "not yours".

tested on amd64 as working. written for arm64 testing.

show more ...

- avoid using globals where it is simple.
- simplify: only use stdio for output.

factor out rawname() from dump, fsck, savecore.

print the missing flag if missing argument...

- always print corrupted dump messages instead of silently failing
- don't require /var/crash to be specified, default to it
- determine and print the kernel name
- print all messages to syslog

static + __dead

Instead of exiting with an obscure error message if -N /kernelname isn't
specified, blithely assume the kernel will consume around 20 megs.

fix sign-compare issues

Fix savecore so it uses /dev/ksyms when it can (doesn't override libkvm
default). Fixes misc/38425.

fix dev_t format.

Enable the faked ELF kernel by default, now that ksyms output works with
gdb. Makes it possible to do post-mortem debugging on a modular kernel.

- Add code to produce a fake ELF kernel containing only symbol info. Done
by extracting ksyms state from the dump, so we have symbols for modules
loaded at the time of the crash. If ksyms extract

- Add code to produce a fake ELF kernel containing only symbol info. Done
by extracting ksyms state from the dump, so we have symbols for modules
loaded at the time of the crash. If ksyms extraction fails, this falls
back to the current behaviour of copying /netbsd.

- Leave it disabled for the moment: (1) we are about to branch (2) most
tools understand the artificial kernel but gdb has trouble with it for
some reason.

show more ...

Explicitly check that the dump device is not the console, a tty or pty.
While the list is adhoc, the problems reported are always with
/dev/console. Adresses PR 38425 and similiar issues with Xen.

Remove the \n and tabs from the __COPYRIGHT() strings.
(Tweak some to use a consistent format.)

sync() a couple of times to benefit diagnosis of sloppy kernel code that
crashes during boot to multiuser.

Don't use the block device to read crashdumps, if possible.

tyop: _time_seconf -> _time_second

from Anon Ymous: remove (void)&var;

c99 initializers

accept either "time_second" or "time" as kernel timevariable in
order to support common userland in enviroments where
timecounter kernels and non timecounter kernel co-exist
(e.g. xen & i386)

support timecounters (struct time gone, use time_second then)

Free some allocated memory in the failure case when trying to
fetch the panic string.

Fixes Coverity CID 1856.