https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=229641

Jilles Tjoelker <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|New                         |Open

--- Comment #3 from Jilles Tjoelker <[email protected]> ---
This looks like a valid bug.

This change should have some tests. The printf utility already has tests so
adding tests for this bug seems appropriate.

The copy of the format string is unnecessary as `start` is already a copy.
Getting rid of it would also fix the bug that there is a memory leak if the
allocation for bfmt succeeds but the allocation for p fails.

(On another note, `start` can be a rather large allocation on the stack.
Ideally, no space would be reserved past the specifier such as b, c, s.)

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "[email protected]"

Reply via email to