projects
/
mmh
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix missing va_end call in uip/mhmisc.c
[mmh]
/
uip
/
pick.c
diff --git
a/uip/pick.c
b/uip/pick.c
index
3ac9e11
..
7d9c086
100644
(file)
--- 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
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);
}
} 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);
} else {
/* if it doesn't match, then unselect it */
unset_selected(mp, msgnum);
- mp->numsel--;
}
if (fp)
fclose(fp);
}
if (fp)
fclose(fp);
@@
-732,7
+731,7
@@
pattern: ;
padvise(NULL, "pattern error in %s %s", argp[-2], cp);
return NULL;
}
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:
return n;
case PROTHR:
@@
-769,8
+768,7
@@
newnexus(int (*action)())
{
struct nexus *p;
{
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;
p->n_action = action;
return p;
@@
-1251,28
+1249,32
@@
plist
fseek(fp, start, SEEK_SET);
for (state = FLD2, bp = NULL;;) {
switch (state = m_getfld2(state, &f, fp)) {
fseek(fp, start, SEEK_SET);
for (state = FLD2, bp = NULL;;) {
switch (state = m_getfld2(state, &f, fp)) {
+ case LENERR2:
+ state = FLD2;
+ /* FALL */
+
case FLD2:
if (bp) {
case FLD2:
if (bp) {
- free(bp);
- bp = NULL;
+ mh_free0(&bp);
}
}
- bp = getcpy(f.value);
+ bp = mh_xstrdup(f.value);
if (mh_strcasecmp(f.name, n->n_datef)==0) {
break;
}
continue;
if (mh_strcasecmp(f.name, n->n_datef)==0) {
break;
}
continue;
- case LENERR2:
case FMTERR2:
case FMTERR2:
- case ERR2:
advise(NULL, "format error in message %d", msgnum);
advise(NULL, "format error in message %d", msgnum);
+ state = FLD2;
+ continue;
+
+ case IOERR2:
+ adios(EX_IOERR, "m_getfld2", "io error on message %d", msgnum);
/* FALL */
case BODY2:
case FILEEOF2:
/* FALL */
case BODY2:
case FILEEOF2:
- if (bp) {
- free(bp);
- }
+ mh_free0(&bp);
return 0;
default:
return 0;
default:
@@
-1289,6
+1291,6
@@
plist
: (twsort(tw, &n->n_tws) < 0);
if (bp != NULL)
: (twsort(tw, &n->n_tws) < 0);
if (bp != NULL)
- free(bp);
+ mh_free0(&bp);
return state;
}
return state;
}