LP#1974195: fix memory leak when performing fleshed Fieldmapper search
authorGalen Charlton <gmc@equinoxOLI.org>
Thu, 19 May 2022 14:00:48 +0000 (10:00 -0400)
committerJason Stephenson <jason@sigio.com>
Tue, 31 May 2022 14:33:31 +0000 (10:33 -0400)
commitf79400864188583bf0a98ebc961ae04ba33595fa
tree31668b3cdea2fef8acabd7b75d5e92f45f90ebe8
parent60e1bd4602d449f5bfd35a6108c5cccd69c04127
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>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Open-ILS/src/c-apps/oils_sql.c