Ticket #2120 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

Memory Leak in msSetup()

Reported by: woodbri Owned by: sdlime
Priority: normal Milestone: 5.0 release
Component: MapServer C Library Version: 5.0
Severity: normal Keywords:
Cc:

Description

In maputil.c, in msSetup() we call:

#ifdef USE_GD_FT
  if (gdFontCacheSetup() != 0) {
    return MS_FAILURE;
   }
#endif

We need to add a corresponding block in msCleanup() to to free this like:

#ifdef USE_GD_FT
  if (gdFontCacheShutdown() != 0) {
    return MS_FAILURE;
   }
#endif

Change History

Changed 6 years ago by hobu

Steve,

I applied this patch in r6188.

Howard

Changed 6 years ago by woodbri

Howard,

Sorry, my bad. The patch should be as msCleanup() does not return a value.

#ifdef USE_GD_FT
  gdFontCacheShutdown();
#endif

Changed 6 years ago by hobu

  • status changed from new to closed
  • resolution set to fixed

ok, fixed in r6189.

Note: See TracTickets for help on using tickets.