summaryrefslogtreecommitdiffstats
path: root/lib/libc/stdio (follow)
Commit message (Collapse)AuthorAgeFilesLines
* fwide() does not unlock if error was occurred.asou2019-12-031-2/+4
| | | | ok guenther@ and deraadt
* more Version 1 AT&T UNIX history:schwarze2019-09-071-3/+7
| | | | | a few cases that weren't altogether straightforward; tweak and OK jmc@, OK sobrado@
* .Dt same as filenamederaadt2019-08-301-3/+3
|
* mop up stdarg rename; ok deraadtjmc2019-08-304-12/+12
|
* two more syscall == -1 checksderaadt2019-06-291-3/+3
|
* Specify that {v,}asprintf(3) returns precisely -1 on failure,schwarze2019-06-281-7/+11
| | | | | | | | | and that the ret pointer is either unchanged or set to NULL in this case. Since these two functions are not standardized by POSIX, documenting the actual behaviour is the way to go, and the above matches all non-buggy implementations we are aware of. OK millert@ deraadt@
* When system calls indicate an error they return -1, not some arbitraryderaadt2019-06-285-11/+11
| | | | | | value < 0. errno is only updated in this case. Change all (most?) callers of syscalls to follow this better, and let's see if this strictness helps us in the future.
* Simplify the description of [v]snprintf(3), move the descriptionschwarze2019-06-271-19/+18
| | | | | | of the return values to RETURN VALUES, deprecate [v]sprintf(3) and fix a punctuation typo. Joint work with and OK millert@.
* an -> a;jmc2019-06-261-2/+2
|
* The POSIX-compatible way of checking for {v,}{f,s,sn,d}printf(3)deraadt2019-06-261-4/+5
| | | | | failure is with < 0, not the more specific -1 from C discussed at length with millert, nicm, schwarze
* More consistently put remarks about the less useful LC_* categoties,schwarze2019-05-161-3/+13
| | | | | | i.e. those other than LC_CTYPE, into the CAVEATS section, and standardize wording somewhat. OK jmc@
* Fix a comparison in open_memstream not to confuse when a negativeyasuoka2019-05-021-3/+3
| | | | | | value is given for the off. found by nagasaka at IIJ. ok deraadt
* Undo changes to tmpfile.c r1.5.martijn2019-04-262-26/+5
| | | | | | | | | | | | Doing the fchown call causes pledge("tmppath") to be insufficient and the the umask dance may cause race-conditions in multithreaded applications. Also POSIX states the following nowadays: implementations may restrict the permissions, either by clearing the file mode bits or setting them to the value S_IRUSR | S_IWUSR. Encouraging words from tedu@ Standards verification and OK millert@
* in vdprintf(), no need to use the file locking mecanism when usingsemarie2019-03-031-2/+2
| | | | | | | fflush() as the variable is stack based (no possible concurrent access). call directly __sflush() ok visa@ deraadt@
* I am retiring my old email address; replace it with my OpenBSD one.millert2019-01-253-6/+6
|
* For all functions known to be infected by LC_NUMERIC, add shortschwarze2019-01-164-29/+30
| | | | | | | | CAVEATS pointing to the new CAVEATS section in setlocale(3). Make those in wprintf(3) and wscanf(3) more concise since duplicate information is a bad idea. Incompleteness of information originally pointed out by millert@. OK millert@
* Similar CAVEATS regarding LC_NUMERIC as was just committedschwarze2019-01-111-6/+19
| | | | | to wprintf(3) with OK cheloha@ tedu@; also triggered by a smaller diff from Jan Stary <hans at stare dot cz>.
* Clarify that OpenBSD ignores the dangerous category LC_NUMERIC,schwarze2019-01-111-16/+30
| | | | | | and explain best practice for portable programs below CAVEATS. Triggered by a smaller diff from Jan Stary <hans at stare dot cz>. Emphatic OKs from cheloha@ and tedu@.
* Restore the optimization for unbuffered I/O. The buffer needs tomillert2018-12-161-1/+31
| | | | | be reset before each call to __srefill(). Passes new regress. OK semarie@
* Back out rev 1.17 for now, it causes issues with python when buildingmillert2018-12-161-31/+1
| | | | databases/tdb from ports.
* Revisit the optimization for unbuffered I/O. We can use the buffermillert2018-12-141-1/+31
| | | | | | passed to fread(3) directly in the FILE * and call __srefill() in a loop without the memcpy(). This preserves the expected behavior in all cases. OK semarie@, "This is neat" tedu@
* Back out the optimization in rev 1.13, it does not update flags onmillert2018-12-141-16/+1
| | | | EOF or error. This caused a regression in the cPickle python extension.
* some grammar fixes; from dholland@netbsd, -r1.68jmc2018-01-021-6/+6
|
* Fix the return value of fwscanf(3) when encountering an early matchingkevlo2017-12-081-2/+2
| | | | | | failure. This change brings fwscanf(3) back in line with fscanf(3). From FreeBSD; ok deraadt@, millert@
* Consistently .Xr the corresponding wide char functions from char- andschwarze2017-12-0111-26/+37
| | | | | string-handling <stdio.h> functions, like we already do it for <string.h>. Includes a smaller patch from <kshe59 at zoho dot eu>, OK jmc@.
* add missing argument name; from <kshe59 at zoho dot eu>; OK jmc@;schwarze2017-12-011-8/+32
| | | | while here, consistently use .Fo to cure execessive line lengths
* GNU ld has prefixed the contents of .gnu.warning.SYMBOL sectionstb2017-11-285-10/+10
| | | | | | | | with "warning: " since 2003, so the messages themselves need not contain the prefix anymore. From Scott Cheloha ok jca, deraadt
* Use a simple forward search to find '%' in the format string instead oftb2017-11-211-35/+12
| | | | | | | | | | | | | | | | | | | | using mbrtowc(3). Thus, we now treat the format string as a bytestring, not as a multibyte character string. We think that ANSI C made a small error when adding wide characters: The committees essentially replaced "characters" with "wide characters" in the existing printf documentation, which was written before the concept of processing was established. Doing processing on the format string would break some 8-bit format strings in the wild, and that isn't something these committees gave themselves license to do. Based on the "10x printf speedup" commit from android found by tedu: https://github.com/aosp-mirror/platform_bionic/commit/5305a4d4a723b06494b93f2df81733b83a0c46d3 Thanks to millert and schwarze for digging into the history and testing *printf behavior on other platforms. ok deraadt, millert
* Add error checking to some calls to __find_arguments(). Matches similartb2017-11-161-6/+16
| | | | | | | | changes by schwarze to vfprintf.c r1.71. Cherrypicked from android: https://github.com/aosp-mirror/platform_bionic/commit/5305a4d4a723b06494b93f2df81733b83a0c46d3 ok millert
* add missing HISTORY; based on CVS logs and release announcementsschwarze2017-10-171-2/+7
|
* fmt0 is a wchar_t *, so use %ls to reportderaadt2017-08-151-3/+3
|
* Favor err() over perror() in example.anton2017-07-221-3/+3
| | | | ok schwarze@
* 1. mild deprecation noticeschwarze2017-07-041-3/+11
| | | | | | | | 2. point to getline (suggested by nicm@) 3. cross reference fgetc(3) rather than putc(3) 4. add missing error handling to the example code OK nicm@
* Add dprintf() and vdprintf() RETURN VALUES. OK deraadt@millert2017-06-121-2/+4
|
* Use recallocarray in getdelim/getline to clear memory on buffer resizes,brynet2017-04-131-2/+2
| | | | | | inspired by a similar change to fgetln. ok deraadt millert
* Use recallocarray() to avoid leaving detritus in memory when resizingmillert2017-03-171-2/+3
| | | | the string buffer used by asprintf() and vasprintf(). OK deraadt@
* Use recallocarray() to avoid leaving detritus in memory when resizingderaadt2017-03-174-8/+12
| | | | | | buffers. We don't bother doing this for objects containing pointers, but focus on controllable data. ok millert
* Only reallocate the buffer to fit for medium-size allocations wheremillert2017-03-162-18/+28
| | | | | we expanded the buffer to a single page. The final realloc() can be expensive for large buffers and is not realled needed. OK deraadt@
* When reallocating the buffer for asprintf(), just round up to themillert2017-03-161-6/+5
| | | | | nearest page instead of doubling the old size until it is large enough. OK deraadt@
* Use a macro for the initial length of the buffer instead of 127; OK deraadt@millert2017-03-142-6/+10
|
* include float.h for the LDBL_MAX_EXP cpp test in floatio.hjsg2016-10-302-2/+4
|
* Delete casts to off_t and size_t that are implied by assignmentsguenther2016-09-2114-42/+39
| | | | | | | or prototypes. Ditto for some of the char* and void* casts too. verified no change to instructions on ILP32 (i386) and LP64 (amd64) ok natano@ abluhm@ deraadt@ millert@
* Store the return value of mbrtowc() in a size_t, not int.millert2016-08-291-13/+15
| | | | OK schwarze@
* Stop recommending the non-standard and slightly dangerous fgetln(3).schwarze2016-08-271-4/+4
| | | | Recommend POSIX getline(3) instead.
* When a precision is specified for a string format use strnlen()millert2016-08-271-17/+7
| | | | to determine the length instead of doing it manually. OK schwarze@
* improve revision 1.2: in unusual cases, fgetwc(3) can succeedschwarze2016-08-271-2/+12
| | | | | | even though ferror(3) is already set; also from Andrey Chernov <ache at freebsd dot org>; OK millert@
* After read errors, fgetln(3) sometimes succeeded (returning non-NULL)schwarze2016-08-251-3/+6
| | | | | | | | | | | | and failed (setting errno and ferror(3)) both at the same time. That's a bad idea in general, and here in particular since returning partial lines was neither reliable (sometimes, you got NULL anyway) nor predictable (almost always, the line would be truncated long before the actual read error). Instead, on read failure, fail properly and always return NULL. Issue found in a discussion with Andrey Chernov <ache at freebsd dot org> who finally agreed to move FreeBSD into the same direction. The fix is joint work with and OK by millert@.
* set the error indicator on malloc(3) failure;schwarze2016-08-241-2/+4
| | | | | from Andrey Chernov <ache at freebsd dot org>; OK millert@
* bugfix: when fgetwc(3) fails, fgetwln(3) must fail as well;schwarze2016-08-211-2/+2
| | | | OK jca@ martijn@ millert@
* % is escaped with more %, not backslash.tedu2016-08-172-6/+6
|
Copyright © 1996 – 2023 Jason A. Donenfeld. All Rights Reverse Engineered.