comparison src/share/tools/launcher/wildcard.c @ 9058:bad3bed4b323

8006006: [parfait] Memory leak at hotspot/src/share/tools/launcher/wildcard.c Summary: a simple fix to add FileList_free(fl) before returning NULL. Reviewed-by: zgu, coleenp, minqi
author ccheung
date Fri, 29 Mar 2013 14:18:40 -0700
parents cb2d0a362639
children
comparison
equal deleted inserted replaced
9057:8554c55669b0 9058:bad3bed4b323
366 wildcardFileList(const char *wildcard) 366 wildcardFileList(const char *wildcard)
367 { 367 {
368 const char *basename; 368 const char *basename;
369 FileList fl = FileList_new(16); 369 FileList fl = FileList_new(16);
370 WildcardIterator it = WildcardIterator_for(wildcard); 370 WildcardIterator it = WildcardIterator_for(wildcard);
371 if (it == NULL) 371 if (it == NULL) {
372 FileList_free(fl);
372 return NULL; 373 return NULL;
374 }
373 while ((basename = WildcardIterator_next(it)) != NULL) 375 while ((basename = WildcardIterator_next(it)) != NULL)
374 if (isJarFileName(basename)) 376 if (isJarFileName(basename))
375 FileList_add(fl, wildcardConcat(wildcard, basename)); 377 FileList_add(fl, wildcardConcat(wildcard, basename));
376 WildcardIterator_close(it); 378 WildcardIterator_close(it);
377 return fl; 379 return fl;