# include "config.h"
#endif
+#include <lib-lib/mem.h>
+#include <lib-lib/str.h>
+#include <lib-lib/macros.h>
+#include <lib-lib/buffer.h>
+#include <lib-lib/file.h>
+
#include "mutt.h"
-#include "buffer.h"
#include "mx.h"
#include "mutt_crypt.h"
#include "compress.h"
#endif
-#include "lib/mem.h"
-#include "lib/intl.h"
-#include "lib/str.h"
#include "lib/rx.h"
#include <limits.h>
#include <ctype.h>
#include <unistd.h>
+#define ERROR_STOP 0
+
typedef struct hook {
int type; /* hook type */
rx_t rx; /* regular expression */
pattern_t *pat = NULL;
char path[_POSIX_PATH_MAX];
- memset (&pattern, 0, sizeof (pattern));
- memset (&command, 0, sizeof (command));
+ p_clear(&pattern, 1);
+ p_clear(&command, 1);
if (*s->dptr == '!') {
s->dptr++;
if (data & (M_FOLDERHOOK | M_MBOXHOOK)) {
strfcpy (path, pattern.data, sizeof (path));
_mutt_expand_path (path, sizeof (path), 1);
- mem_free (&pattern.data);
- memset (&pattern, 0, sizeof (pattern));
- pattern.data = str_dup (path);
+ p_delete(&pattern.data);
+ p_clear(&pattern, 1);
+ pattern.data = m_strdup(path);
}
#ifdef USE_COMPRESSED
else if (data & (M_APPENDHOOK | M_OPENHOOK | M_CLOSEHOOK)) {
strfcpy (tmp, pattern.data, sizeof (tmp));
mutt_check_simple (tmp, sizeof (tmp), DefaultHook);
- mem_free (&pattern.data);
- memset (&pattern, 0, sizeof (pattern));
- pattern.data = str_dup (tmp);
+ p_delete(&pattern.data);
+ p_clear(&pattern, 1);
+ pattern.data = m_strdup(tmp);
}
if (data & (M_MBOXHOOK | M_SAVEHOOK | M_FCCHOOK)) {
strfcpy (path, command.data, sizeof (path));
mutt_expand_path (path, sizeof (path));
- mem_free (&command.data);
- memset (&command, 0, sizeof (command));
- command.data = str_dup (path);
+ p_delete(&command.data);
+ p_clear(&command, 1);
+ command.data = m_strdup(path);
}
/* check to make sure that a matching hook doesn't already exist */
for (ptr = Hooks; ptr; ptr = ptr->next) {
if (ptr->type == data &&
- ptr->rx.not == not && !str_cmp (pattern.data, ptr->rx.pattern)) {
+ ptr->rx.not == not && !m_strcmp(pattern.data, ptr->rx.pattern)) {
if (data &
(M_FOLDERHOOK | M_SENDHOOK | M_SEND2HOOK | M_MESSAGEHOOK |
M_ACCOUNTHOOK | M_REPLYHOOK)) {
/* these hooks allow multiple commands with the same
* pattern, so if we've already seen this pattern/command pair, just
* ignore it instead of creating a duplicate */
- if (!str_cmp (ptr->command, command.data)) {
- mem_free (&command.data);
- mem_free (&pattern.data);
+ if (!m_strcmp(ptr->command, command.data)) {
+ p_delete(&command.data);
+ p_delete(&pattern.data);
return 0;
}
}
* order of execution of the hooks, which i think is desirable since
* a common action to perform is to change the default (.) entry
* based upon some other information. */
- mem_free (&ptr->command);
+ p_delete(&ptr->command);
ptr->command = command.data;
- mem_free (&pattern.data);
+ p_delete(&pattern.data);
return 0;
}
}
goto error;
}
else {
- rx = mem_malloc (sizeof (regex_t));
+ rx = p_new(regex_t, 1);
#ifdef M_CRYPTHOOK
if ((rc =
REGCOMP (rx, NONULL (pattern.data),
{
regerror (rc, rx, err->data, err->dsize);
regfree (rx);
- mem_free (&rx);
+ p_delete(&rx);
goto error;
}
}
if (ptr) {
- ptr->next = mem_calloc (1, sizeof (HOOK));
+ ptr->next = p_new(HOOK, 1);
ptr = ptr->next;
}
else
- Hooks = ptr = mem_calloc (1, sizeof (HOOK));
+ Hooks = ptr = p_new(HOOK, 1);
ptr->type = data;
ptr->command = command.data;
ptr->pattern = pat;
return 0;
error:
- mem_free (&pattern.data);
- mem_free (&command.data);
+ p_delete(&pattern.data);
+ p_delete(&command.data);
return (-1);
}
static void delete_hook (HOOK * h)
{
- mem_free (&h->command);
- mem_free (&h->rx.pattern);
+ p_delete(&h->command);
+ p_delete(&h->rx.pattern);
if (h->rx.rx) {
regfree (h->rx.rx);
}
mutt_pattern_free (&h->pattern);
- mem_free (&h);
+ p_delete(&h);
}
/* Deletes all hooks of type ``type'', or all defined hooks if ``type'' is 0 */
{
while (MoreArgs (s)) {
mutt_extract_token (buf, s, 0);
- if (str_cmp ("*", buf->data) == 0) {
+ if (m_strcmp("*", buf->data) == 0) {
if (current_hook_type) {
snprintf (err->data, err->dsize,
_("unhook: Can't do unhook * from within a hook."));
err.data = buf;
err.dsize = sizeof (buf);
- memset (&token, 0, sizeof (token));
+ p_clear(&token, 1);
for (; tmp; tmp = tmp->next) {
if (!tmp->command)
continue;
if ((regexec (tmp->rx.rx, path, 0, NULL, 0) == 0) ^ tmp->rx.not) {
if (mutt_parse_rc_line (tmp->command, &token, &err) == -1) {
mutt_error ("%s", err.data);
- mem_free (&token.data);
mutt_sleep (1); /* pause a moment to let the user see the error */
- current_hook_type = 0;
- return;
+ if (ERROR_STOP) {
+ p_delete(&token.data);
+ current_hook_type = 0;
+ return;
+ }
}
}
}
}
- mem_free (&token.data);
+ p_delete(&token.data);
current_hook_type = 0;
}
err.data = buf;
err.dsize = sizeof (buf);
- memset (&token, 0, sizeof (token));
+ p_clear(&token, 1);
for (hook = Hooks; hook; hook = hook->next) {
if (!hook->command)
continue;
if (hook->type & type)
if ((mutt_pattern_exec (hook->pattern, 0, ctx, hdr) > 0) ^ hook->rx.not)
if (mutt_parse_rc_line (hook->command, &token, &err) != 0) {
- mem_free (&token.data);
mutt_error ("%s", err.data);
mutt_sleep (1);
- current_hook_type = 0;
- return;
+ if (ERROR_STOP) {
+ p_delete(&token.data);
+ current_hook_type = 0;
+ return;
+ }
}
}
- mem_free (&token.data);
+ p_delete(&token.data);
current_hook_type = 0;
}
(env->to || env->cc || env->bcc)) {
adr = env->to ? env->to : (env->cc ? env->cc : env->bcc);
mutt_safe_path (buf, sizeof (buf), adr);
- mutt_concat_path (path, NONULL (Maildir), buf, pathlen);
+ mutt_concat_path(path, pathlen, NONULL(Maildir), buf);
if (!option (OPTFORCENAME) && mx_access (path, W_OK) != 0)
strfcpy (path, NONULL (Outbox), pathlen);
}
err.data = buf;
err.dsize = sizeof (buf);
- memset (&token, 0, sizeof (token));
+ p_clear(&token, 1);
for (hook = Hooks; hook; hook = hook->next) {
if (!(hook->command && (hook->type & M_ACCOUNTHOOK)))
if ((regexec (hook->rx.rx, url, 0, NULL, 0) == 0) ^ hook->rx.not) {
if (mutt_parse_rc_line (hook->command, &token, &err) == -1) {
- mem_free (&token.data);
mutt_error ("%s", err.data);
mutt_sleep (1);
-
- return;
+ if (ERROR_STOP) {
+ p_delete(&token.data);
+ current_hook_type = 0;
+ return;
+ }
}
}
}
- mem_free (&token.data);
+ p_delete(&token.data);
}
#endif