[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: simple valgrind test found what appear to be serious memory problems
From: |
Thien-Thi Nguyen |
Subject: |
Re: simple valgrind test found what appear to be serious memory problems |
Date: |
Tue, 05 Oct 2010 00:14:51 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux) |
() Paul Eggert <address@hidden>
() Thu, 30 Sep 2010 09:21:35 -0700
by a careful code audit.
I have changed my practice a (tiny) bit. The non-initializing
allocation function ‘alloc’ (b-divvy.c) now includes a warning. :-)
More seriously, i have reviewed the complete fanout of ‘alloc’ and
found/fixed bugs where structures (using the allocation) are not fully
initialized, as well as other memory-related bugs.
The (abridged) list of changes in the repo since 5.7.93 is now:
$ glog 5.7.93.. --reverse
[...]
f1383af [maint] Do "gnulib-tool --add-import" for bootstrap.
dc2ff9d [maint] Simplify ‘PATHPREFIX’ Makefile-internal protocol.
db5a008 [int] Use ‘struct symdef’ array as "three manifestations" ‘merge’ arg.
140ab2d [maint] Let the configure script instantiate rcsfreeze.
c269dd3 [v] Use Automake for test preparation.
b4ae77b [int] Avoid identifier ‘SHARED’ (Solaris 8 header conflict).
44060ab [int] Fix bug: Save notyet objects in caller space.
7489411 [build] Delete unused macro.
a231f5c [int] Fix bug: Fully initialize notyet objects.
154cc2b [int] Fix bug: Fully initialize delta objects.
df3b780 [v] Use Automake for building xorlf.
c6dfec5 [v] Do stdout/stderr split in shell func, not in makefile.
f607093 [v] Make test wd in common; arrange for cleanup there, too.
fcd6d46 [v] Use Automake for generating top-level test driver.
fed95b9 [int] Fix bug: Initialize notyet ‘.branches’ correctly.
f6d4053 [v] Delete unused shell func.
f3de5d2 [v] Centralize shell vars ‘w’, ‘v’.
At the moment, i am overhauling tests/, trying to decide whether or not
to test against libparts.a (via src/yarrrr.c repurposing). WDYT?