CVS log for src/sys/lib/libsa/subr_prf.c

Up to [cvs.NetBSD.org] / src / sys / lib / libsa

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

Only a leading '0' in a format width marks zero-padding, otherwise
it's a regular digit.

Pull up following revision(s) (requested by rin in ticket #1636):

	sys/lib/libsa/subr_prf.c: revision 1.30

libsa/printf: Do not fetch long va_arg as long long.

This does real harm iff all of the following conditions are satisfied:
(1) On ILP32 architectures.
(2) Both LIBSA_PRINTF_LONGLONG_SUPPORT and LIBSA_PRINTF_WIDTH_SUPPORT
    compile-time options are enabled.
(3) Width field is used with 'l' modifier.

This is an implicit-fallthrough bug, but unfortunately, GCC 10.4 cannot
find this out somehow...

Pull up following revision(s) (requested by rin in ticket #185):

	sys/lib/libsa/subr_prf.c: revision 1.30

libsa/printf: Do not fetch long va_arg as long long.

This does real harm iff all of the following conditions are satisfied:
(1) On ILP32 architectures.
(2) Both LIBSA_PRINTF_LONGLONG_SUPPORT and LIBSA_PRINTF_WIDTH_SUPPORT
    compile-time options are enabled.
(3) Width field is used with 'l' modifier.

This is an implicit-fallthrough bug, but unfortunately, GCC 10.4 cannot
find this out somehow...

libsa/printf: Do not fetch long va_arg as long long.

This does real harm iff all of the following conditions are satisfied:

(1) On ILP32 architectures.
(2) Both LIBSA_PRINTF_LONGLONG_SUPPORT and LIBSA_PRINTF_WIDTH_SUPPORT
    compile-time options are enabled.
(3) Width field is used with 'l' modifier.

This is an implicit-fallthrough bug, but unfortunately, GCC 10.4 cannot
find this out somehow...

XXX
Pull up to netbsd-10 and netbsd-9. netbsd-8 is not affected.

Make libsa's vsnprintf() work as expected when passed a NULL
destinatino buffer.

Sync with HEAD

- remove unreachable code

update from HEAD

Revert stupid and untested changes. There was no extra copy in the macros.

Christos, you should reread our commit guideline as a Core member.

remove extra copy of the macro.

Rename padding macros to reflect and clarify the original intention.

No binary changes on luna68k (which has -DLIBSA_PRINTF_WIDTH_SUPPORT).

Rebase to HEAD as of a few days ago.

sync with head.

for a reference, the tree before this commit was tagged
as yamt-pagecache-tag8.

this commit was splitted into small chunks to avoid
a limitation of cvs.  ("Protocol error: too many arguments")

sync with head

for LIBSA_PRINTF_WIDTH_SUPPORT,
Fix right and left-adjusting padding.

intmax_t might be long long, handle accordingly. from christos

Add 'j' format modifier for intmax_t.

Retire varargs.h support. Move machine/stdarg.h logic into MI
sys/stdarg.h and expect compiler to provide proper builtins, defaulting
to the GCC interface. lint still has a special fallback.
Reduce abuse of _BSD_VA_LIST_ by defining __va_list by default and
derive va_list as required by standards.

Sync with HEAD.

sync with head

KNF a bit.

sync with head

Sync with HEAD

No trailing 0 byte for hexdigits.

Move HEXDIGIT to mvme68k's sboot. It seems to be the only user.
Save a byte by explicitly specifying the size.

sync with head

Add long long (%lld etc.) support and width (%02x etc.) support
in libsa printf(3).  Disabled by default but enabled by
-DLIBSA_PRINTF_LONGLONG_SUPPORT and -DLIBSA_PRINTF_WIDTH_SUPPORT.
Provided by tnozaki@ for my libsa debugging. Thanks!

sync with HEAD

Sync with HEAD.

sync with head

Sync with HEAD. amd64 Xen support needs testing.

style, indent, and ANSI-fy.

sync with head.

sync with head.

/* fall through */ -> /* FALLTHROUGH */

Support 'z' (size_t) and 't' (ptrdiff_t) specifiers.

XXX: ptrdiff_t is only ever defined in <stddef.h> and is defined
unconditionally, without the ifndef dance.  Until we have an expert
opinion, abuse intptr_t for the 't' size check.

merge ktrace-lwp.

Sync with HEAD. Here we go again...

Provide HEXDIGITS as well (needed on mvme68k/stand)

Yes, it was a cool trick >20 years ago to use "0123456789abcdef"[a] to
implement, xtoa(), but I think defining the samestring 50 times is a bit
too much. Defined HEXDIGITS and hexdigits in subr_prf.c and use it...

Fix the sync with head I botched.

Sync with HEAD.

Sync with HEAD

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22364, verified by myself.

Remove two unused variable to fix a build problem.

Remove %b support.
Encouraged by thorpej.

If the format string ends with "%" or "%l", print those characters, just
as we would if they were part of an invalid conversion specification.
Code-size neutral on ARM; not tested elsewhere.

ANSIfy, assume __STDC__, un-__P, KNF.
Generated code unchanged.

Catch up with -current.

Catch up to -current.

Add a missing "static".

Sync kqueue branch with -current.

Catch up with -current.

Catch up to -current.
Again.

va_{start,end} audit:
Make sure that each va_start has one and only one matching va_end,
especially in error cases.
If the va_list is used multiple times, do multiple va_starts/va_ends.
If a function gets va_list as argument, don't let it use va_end (since
it's the callers responsibility).

Improved by comments from enami and christos -- thanks!

Heimdal/krb4/KAME changes already fed back, rest to follow.

Inspired by, but not not based on, OpenBSD.

Update thorpej_scsipi to -current as of a month ago

Kill some more register declarations.

Sync w/ -current.

set lflag when printing pointers with %p

Split the printf variants into separate files.