From f79400864188583bf0a98ebc961ae04ba33595fa Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Thu, 19 May 2022 10:00:48 -0400 Subject: [PATCH] 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 Signed-off-by: Jason Stephenson --- Open-ILS/src/c-apps/oils_sql.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/Open-ILS/src/c-apps/oils_sql.c b/Open-ILS/src/c-apps/oils_sql.c index 42a846a..93f2577 100644 --- a/Open-ILS/src/c-apps/oils_sql.c +++ b/Open-ILS/src/c-apps/oils_sql.c @@ -6388,8 +6388,6 @@ static jsonObject* doFieldmapperSearch( osrfMethodContext* ctx, osrfHash* class_ osrfLogDebug( OSRF_LOG_MARK, "Fleshing of %s complete", osrfHashGet( kid_link, "field" ) ); - osrfLogDebug( OSRF_LOG_MARK, "%s", jsonObjectToJSON( cur )); - } // end while loop traversing list of fleshable fields } -- 1.7.2.5