Bug 2833 – doaccept() directly free()'s connection rather than freeconn()'ing it

Bug 2833 - doaccept() directly free()'s connection rather than freeconn()'ing it

Summary:

doaccept() directly free()'s connection rather than freeconn()'ing it

Status:	RESOLVED FIXED

Product:	Replica Location
Component:	RLS
Version:	development
Platform:	PC Linux

Importance:	P3 normal
Target Milestone:	---
Assigned To:	Rob S

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2005-03-01 13:29 by Rob S
Modified:	2005-03-29 02:21 (History)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Rob S 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.