X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/6b4bc4a25eaac2eddd1bef51e3f00c21f98ed32d..01416e0e4c7e5b6ba8a1a1bf6eb0d8638f32f069:/sbr/seq_save.c

diff --git a/sbr/seq_save.c b/sbr/seq_save.c
index 3c53e4bc..5b06dcdf 100644
--- a/sbr/seq_save.c
+++ b/sbr/seq_save.c
@@ -24,15 +24,19 @@
 void
 seq_save (struct msgs *mp)
 {
-    int i;
+    size_t i;
     char flags, *cp, attr[BUFSIZ], seqfile[PATH_MAX];
     FILE *fp;
     sigset_t set, oset;
 
     /* check if sequence information has changed */
     if (!(mp->msgflags & SEQMOD)) {
-    	if (mp->seqhandle)
+    	if (mp->seqhandle) {
 	    lkfclosedata (mp->seqhandle, mp->seqname);
+	    mp->seqhandle = NULL;
+	    free(mp->seqname);
+	    mp->seqname = NULL;
+	}
 	return;
     }
     mp->msgflags &= ~SEQMOD;
@@ -50,11 +54,12 @@ seq_save (struct msgs *mp)
     else
 	snprintf (seqfile, sizeof(seqfile), "%s/%s", mp->foldpath, mh_seq);
 
-    for (i = 0; mp->msgattrs[i]; i++) {
-	snprintf (attr, sizeof(attr), "atr-%s-%s", mp->msgattrs[i], mp->foldpath);
+    for (i = 0; i < svector_size (mp->msgattrs); i++) {
+	snprintf (attr, sizeof(attr), "atr-%s-%s",
+		  svector_at (mp->msgattrs, i), mp->foldpath);
 
 	/* get space separated list of sequence ranges */
-	if (!(cp = seq_list(mp, mp->msgattrs[i]))) {
+	if (!(cp = seq_list(mp, svector_at (mp->msgattrs, i)))) {
 	    context_del (attr);			/* delete sequence from context */
 	    continue;
 	}
@@ -86,7 +91,7 @@ priv:
 		    rewind(fp);
 		    ftruncate(fileno(fp), 0);
 		} else if ((fp = lkfopendata (seqfile, "w")) == NULL
-			&& (unlink (seqfile) == -1 ||
+			&& (m_unlink (seqfile) == -1 ||
 			    (fp = lkfopendata (seqfile, "w")) == NULL)) {
 		    admonish (attr, "unable to write");
 		    goto priv;
@@ -100,7 +105,7 @@ priv:
 		sigaddset(&set, SIGTERM);
 		sigprocmask (SIG_BLOCK, &set, &oset);
 	    }
-	    fprintf (fp, "%s: %s\n", mp->msgattrs[i], cp);
+	    fprintf (fp, "%s: %s\n", svector_at (mp->msgattrs, i), cp);
 	}
     }
 
@@ -113,7 +118,7 @@ priv:
 	 * public sequences, then remove that file.
 	 */
 	if (!is_readonly(mp))
-	    unlink (seqfile);
+	    (void) m_unlink (seqfile);
     }
 
     /*