Bug 2833

Summary: doaccept() directly free()'s connection rather than freeconn()'ing it
Product: Replica Location Reporter: Rob S <schuler@isi.edu>
Component: RLSAssignee: Rob S <schuler@isi.edu>
Status: RESOLVED FIXED    
Severity: normal CC: annc@isi.edu, shishir@isi.edu
Priority: P3    
Version: development   
Target Milestone: ---   
Hardware: PC   
OS: Linux   

Description From 2005-03-01 13:29:51 
The RLS (server.c:662) directly free's the connection variable rather than
calling freeconn() to properly release it. In this case it has only a minor
effect on the system -- the actual connection is _not_ open so there is little
loss of resources. However, the connection list is not properly corrected and
the connection count is not adjusted.

We have not experienced any problem due to this, but it is clear from the code
that it should be corrected. Shishir identified this problem.

Related to this, I have some slight concern about freeconn() itself, which I
will also look into in the process of correcting the above. It is nothing major
though.