src/hg/hgLiftOver/hgLiftOver.c 1.62

1.62 2009/07/14 20:17:30 markd
fixed some missing closes noticed while search for a bug
Index: src/hg/hgLiftOver/hgLiftOver.c
===================================================================
RCS file: /projects/compbio/cvsroot/kent/src/hg/hgLiftOver/hgLiftOver.c,v
retrieving revision 1.61
retrieving revision 1.62
diff -b -B -U 4 -r1.61 -r1.62
--- src/hg/hgLiftOver/hgLiftOver.c	8 May 2009 23:36:25 -0000	1.61
+++ src/hg/hgLiftOver/hgLiftOver.c	14 Jul 2009 20:17:30 -0000	1.62
@@ -419,9 +419,8 @@
     struct tempName oldTn, mappedTn, unmappedTn;
     FILE *old, *mapped, *unmapped;
     char *line;
     int lineSize;
-    struct lineFile *errFile;
     char *fromDb, *toDb;
     int ct = 0, errCt = 0;
 
     /* read in user data and save to file */
@@ -492,21 +491,22 @@
         printf("<A HREF=%s TARGET=_blank>Display failure file</A>&nbsp; &nbsp;\n",
                          unmappedTn.forCgi);
         printf("<A HREF=\"../cgi-bin/hgLiftOver?%s=1\" TARGET=_blank>Explain failure messages</A>\n", HGLFT_ERRORHELP_VAR);
         puts("<P>Failed input regions:\n");
-        fclose(unmapped);
-        errFile = lineFileOpen(unmappedTn.forCgi, TRUE);
+        struct lineFile *errFile = lineFileOpen(unmappedTn.forCgi, TRUE);
         puts("<BLOCKQUOTE><PRE>\n");
         while (lineFileNext(errFile, &line, &lineSize))
             puts(line);
+        lineFileClose(&errFile);
         puts("</PRE></BLOCKQUOTE>\n");
         }
     if (sameString(dataFormat, POSITION_FORMAT) && multiple)
 	{
         puts("<BLOCKQUOTE><PRE>\n");
         puts("Note: multiple checkbox ignored since it is not supported for position format.");
         puts("</PRE></BLOCKQUOTE>\n");
 	}
+    carefulClose(&unmapped);
     }
 webDataFormats();
 webDownloads();
 cartWebEnd();