Bugzilla – Full Text Bug Listing
|Summary:||doaccept() directly free()'s connection rather than freeconn()'ing it|
|Product:||Replica Location||Reporter:||Rob S <firstname.lastname@example.org>|
|Component:||RLS||Assignee:||Rob S <email@example.com>|
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.