diff options
| author | Michael Coleman <tutufan@gmail.com> | 2007-02-27 23:44:42 -0600 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2007-02-27 22:02:41 -0800 |
| commit | 163d7b9b8536d206eda7eb0eccb0fc211812346f (patch) | |
| tree | 3da13237a1d8c01e31c5669a1e24f44c62d610f3 /builtin-fmt-merge-msg.c | |
| parent | a91d49cd369ac5fc8e1a17357a975d09cf6c8cb3 (diff) | |
| download | git-163d7b9b8536d206eda7eb0eccb0fc211812346f.tar.gz | |
builtin-fmt-merge-msg: fix bugs in --file option
If --file's argument is missing, don't crash. If it cannot be opened,
die with an error message.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-fmt-merge-msg.c')
| -rw-r--r-- | builtin-fmt-merge-msg.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/builtin-fmt-merge-msg.c b/builtin-fmt-merge-msg.c index 87d3d63ec7..5be6fb4388 100644 --- a/builtin-fmt-merge-msg.c +++ b/builtin-fmt-merge-msg.c @@ -259,13 +259,15 @@ int cmd_fmt_merge_msg(int argc, const char **argv, const char *prefix) else if (!strcmp(argv[1], "--no-summary")) merge_summary = 0; else if (!strcmp(argv[1], "-F") || !strcmp(argv[1], "--file")) { - if (argc < 2) + if (argc < 3) die ("Which file?"); if (!strcmp(argv[2], "-")) in = stdin; else { fclose(in); in = fopen(argv[2], "r"); + if (!in) + die("cannot open %s", argv[2]); } argc--; argv++; } else |