gnats/504: Re: gnats/504: gnatsd crashes when trying to lock a nonexistent PR

[bug-gnats]

The following reply was made to PR gnats/504; it has been noted by GNATS.

From: Chad Walstrom <address@hidden>
To: address@hidden
Cc: 
Subject: Re: gnats/504: gnatsd crashes when trying to lock a nonexistent PR
Date: Mon, 31 Oct 2005 23:41:57 -0600

 It looks like Mel designed get_pr_path() to be generic, called without
 prior knowledge about the status of the index.  Unless Mel had other
 use cases apparent in his GNATS DBI fork, it might be logical to
 simplify the function.
 
 Your assessment is that by removing the getCategoryFromIndex() call,
 gnatsd is no longer crashing when locking non-existent PR's.  Perhaps
 we should look at that call in depth as well.  It appears only to be
 used by get_pr_path() currently, but since this is an in-progress
 merge, I would like to ask Mel's opinion (or a look at Mel's fork) on
 the matter.  My next block of time to look at this is Friday.
 
 Happy Halloween!
 
 -- 
 Chad Walstrom <address@hidden>           http://www.wookimus.net/
            assert(expired(knowledge)); /* core dump */