LP#1974195: fix memory leak when performing fleshed Fieldmapper search user/gmcharlt/lp1974195_fix_flesh_fm_search_leak
authorGalen Charlton <gmc@equinoxOLI.org>
Thu, 19 May 2022 14:00:48 +0000 (10:00 -0400)
committerGalen Charlton <gmc@equinoxOLI.org>
Thu, 19 May 2022 14:00:48 +0000 (10:00 -0400)
commit6f4e42e96fc5b4e699e2f35095ae15261d4786dd
tree83ce11874141a2d691ef38cace7c854e82545aff
parent83b92e26b525cc48f254f1c3964a752e0ad87a28
LP#1974195: fix memory leak when performing fleshed Fieldmapper search

This patch fixes a memory leak that occurs when performing
a fleshed search of an IDL class via open-ils.cstore, open-ils.pcrud,
or open-ils.rstore. It removes a debug-level log entry that didn't
free a temporary string.

To test
-------
[1] Perform a PCRUD search that includes fleshing and retrieves
    a large responses. One example is a search on bre that fleshes
    call numbers and copies and returns a few hundred bibs.
[2] Observe that the PCRUD drone that serviced the request has
    increased its memory usage significantly.
[3] Apply the patch and repeat step 1. This time, the drone's memory
    usage should not (persistently) increase as much.

Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Open-ILS/src/c-apps/oils_sql.c