X-Git-Url: http://git.marmaro.de/?a=blobdiff_plain;f=uip%2Fpick.c;h=1df59b186f3fae1d7ad09b6a559f581d9a136855;hb=3244935a74559715749a60c03e7b608a227e0438;hp=984d80bdf20bf4581ce47ff78bdc962bd10cac1e;hpb=4dc39c08f07428ff5f39acd7b0ddee30e0a004f6;p=mmh

diff --git a/uip/pick.c b/uip/pick.c
index 984d80b..1df59b1 100644
--- a/uip/pick.c
+++ b/uip/pick.c
@@ -197,7 +197,7 @@ main(int argc, char **argv)
 			if (folder)
 				adios(EX_USAGE, NULL, "only one folder at a time!");
 			else
-				folder = getcpy(expandfol(cp));
+				folder = mh_xstrdup(expandfol(cp));
 		} else
 			app_msgarg(&msgs, cp);
 	}
@@ -274,7 +274,6 @@ main(int argc, char **argv)
 			} else {
 				/* if it doesn't match, then unselect it */
 				unset_selected(mp, msgnum);
-				mp->numsel--;
 			}
 			if (fp)
 				fclose(fp);
@@ -732,7 +731,7 @@ pattern: ;
 			padvise(NULL, "pattern error in %s %s", argp[-2], cp);
 			return NULL;
 		}
-		n->n_patbuf = getcpy(dp);
+		n->n_patbuf = mh_xstrdup(dp);
 		return n;
 
 	case PROTHR:
@@ -769,8 +768,7 @@ newnexus(int (*action)())
 {
 	struct nexus *p;
 
-	if ((p = (struct nexus *) mh_xcalloc((size_t) 1, sizeof *p)) == NULL)
-		adios(EX_OSERR, NULL, "unable to allocate component storage");
+	p = mh_xcalloc(1, sizeof *p);
 
 	p->n_action = action;
 	return p;
@@ -1254,10 +1252,9 @@ plist
 		case FLD:
 		case FLDPLUS:
 			if (bp != NULL) {
-				free(bp);
-				bp = NULL;
+				mh_free0(&bp);
 			}
-			bp = getcpy(buf);
+			bp = mh_xstrdup(buf);
 			while (state == FLDPLUS) {
 				state = m_getfld(state, name, buf,
 						sizeof buf, fp);
@@ -1274,7 +1271,7 @@ plist
 			if (state == LENERR || state == FMTERR)
 				advise(NULL, "format error in message %d", msgnum);
 			if (bp != NULL)
-				free(bp);
+				mh_free0(&bp);
 			return 0;
 
 		default:
@@ -1291,6 +1288,6 @@ plist
 			: (twsort(tw, &n->n_tws) < 0);
 
 	if (bp != NULL)
-		free(bp);
+		mh_free0(&bp);
 	return state;
 }