Mercurial > hg > graal-compiler
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; |