Patch from Scott McKellar:
authormiker <miker@9efc2488-bf62-4759-914b-345cdb29e865>
Thu, 31 Jan 2008 19:07:30 +0000 (19:07 +0000)
committermiker <miker@9efc2488-bf62-4759-914b-345cdb29e865>
Thu, 31 Jan 2008 19:07:30 +0000 (19:07 +0000)
commitec81ff1f23be47947212673ec912d7bdf9a66b32
tree8051105bb788fedcefbc71a51ee3ce0fe7900557
parentbc62fd003fe5d63e8b7c13355019221c38eb4356
Patch from Scott McKellar:

1. I replaced the following identifiers with their camel-case
equivalents:

   osrf_message             ==> osrfMessage
   osrf_message_free        ==> osrfMessageFree
   string_array             ==> osrfStringArray
   string_array_get_string  ==> osrfStringArrayGetString

2. In osrf_app_server_session_init() we were leaking session in the
case of an early return.  I stuck in a free().

3. Likewise in osrfAppSessionMakeLocaleRequest() we were leaking req
in the case of an early return.  I plugged that one too.

4. In osrfAppRequestRespondComplete() we were leaking payload in one
branch of an if/else.  I moved the osrfMessageFree() beyond both
branches so that it would be unconditional.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1235 9efc2488-bf62-4759-914b-345cdb29e865
src/libopensrf/osrf_app_session.c