]> granicus.if.org Git - file/commit
Replace use of (v)snprintf with (v)asprintf. This has several advantages:
authorReuben Thomas <rrt@sc3d.org>
Tue, 19 Feb 2008 00:58:59 +0000 (00:58 +0000)
committerReuben Thomas <rrt@sc3d.org>
Tue, 19 Feb 2008 00:58:59 +0000 (00:58 +0000)
commit6057f6e378adc95e23c4de1d629b066df4c64f5c
treea64d45c6f8609e2da6f93d6e2a5eea776c1b2009
parent09f647647482ab3fc435ba8281cfc713c4d51a70
Replace use of (v)snprintf with (v)asprintf. This has several advantages:

1. (Mostly) easier to program with as the allocation is done for you.
   (Sometimes it's a little more complicated to get the deallocation
   right). The amount of code removed as a result by this commit is
   greater than the amount introduced (excluding the (v)asprintf
   implementation).

2. Safer: no buffer overflows.

3. Works on all platforms: BSD-licensed mature xnprintf code used
   (I've been using this for some years in Zile with no problems). No
   more overflows due to the previous bogus snprintf/vsnprintf
   fallbacks.

Also add and use a MAX() macro in the same vein as MIN().
configure.ac
src/apprentice.c
src/asprintf.c [new file with mode: 0644]
src/file.h
src/funcs.c
src/magic.c
src/softmagic.c
src/vasprintf.c [new file with mode: 0644]