simon-svn: agedu: simon

Commits to Tartarus CVS repository. tartarus-commits at lists.tartarus.org
Mon Oct 29 18:33:46 GMT 2012


SVN root:       svn://svn.tartarus.org/sgt
Changes by:     simon
Revision:       9693
Date:           2012-10-29 18:33:46 +0000 (Mon, 29 Oct 2012)

Log message (9 lines):
Fix a memory access bug in the trie construction. When we attempt to
examine the partially build switch node at a given depth, we must
abort the attempt if that depth is _at least_ tb->switchsize, not just
if it's greater (since, as usual, elements in the array exist up to
but not including tb->switchsize).

This was reported as a segfault by an AIX user recently, but turns out
not to be a platform-specific issue: valgrind confirms that it's wrong
on Linux too, even though it hasn't happened to explode for anyone.

Modified files:
U   agedu/trie.c

Links:
http://svn.tartarus.org/sgt/?rev=9693&view=rev
http://svn.tartarus.org/sgt/agedu/trie.c?rev=9693&r1=9692&r2=9693



More information about the tartarus-commits mailing list