diff --git a/notmuch-tag.c b/notmuch-tag.c index 7d186399..d15f1eda 100644 --- a/notmuch-tag.c +++ b/notmuch-tag.c @@ -203,6 +203,17 @@ notmuch_tag_command (void *ctx, int argc, char *argv[]) break; } if (argv[i][0] == '+' || argv[i][0] == '-') { + if (argv[i][0] == '+' && argv[i][1] == '\0') { + fprintf(stderr, "Error: tag names cannot be empty.\n"); + return 1; + } + if (argv[i][0] == '+' && argv[i][1] == '-') { + /* This disallows adding the non-removable tag "-" and + * enables notmuch tag to take long options in the + * future. */ + fprintf(stderr, "Error: tag names must not start with '-'.\n"); + return 1; + } tag_ops[tag_ops_count].tag = argv[i] + 1; tag_ops[tag_ops_count].remove = (argv[i][0] == '-'); tag_ops_count++; diff --git a/test/tagging b/test/tagging index e4782ed4..980ff927 100755 --- a/test/tagging +++ b/test/tagging @@ -46,4 +46,8 @@ test_expect_equal "$output" "\ thread:XXX 2001-01-05 [1/1] Notmuch Test Suite; One (:\" inbox tag1 unread) thread:XXX 2001-01-05 [1/1] Notmuch Test Suite; Two (inbox tag1 tag4 unread)" +test_expect_code 1 "Empty tag names" 'notmuch tag + One' + +test_expect_code 1 "Tag name beginning with -" 'notmuch tag +- One' + test_done