diff options
| author | Alasdair Kergon <agk@redhat.com> | 2007-01-11 20:11:19 +0000 |
|---|---|---|
| committer | Alasdair Kergon <agk@redhat.com> | 2007-01-11 20:11:19 +0000 |
| commit | 7cc6c5d79a4dad71c63e0197d145c6b1ba855afe (patch) | |
| tree | dca92402841b65b7b5c2323fb1f1eaafb4909219 /daemons/dmeventd/plugins/mirror | |
| parent | 65e635b6857d4e36cf97a2a90d628a7d5cab2ed8 (diff) | |
| download | lvm2-7cc6c5d79a4dad71c63e0197d145c6b1ba855afe.tar.gz lvm2-7cc6c5d79a4dad71c63e0197d145c6b1ba855afe.tar.xz lvm2-7cc6c5d79a4dad71c63e0197d145c6b1ba855afe.zip | |
fail if status args are missing
Diffstat (limited to 'daemons/dmeventd/plugins/mirror')
| -rw-r--r-- | daemons/dmeventd/plugins/mirror/dmeventd_mirror.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c index 9492e3f4..577a8cd3 100644 --- a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c +++ b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c @@ -58,7 +58,7 @@ static int _get_mirror_event(char *params) char *dev_status_str; char *log_status_str; char *sync_str; - char *p; + char *p = NULL; int log_argc, num_devs; /* @@ -70,12 +70,13 @@ static int _get_mirror_event(char *params) if (!dm_split_words(params, 1, 0, &p)) goto out_parse; - num_devs = atoi(p); + if (!(num_devs = atoi(p))) + goto out_parse; p += strlen(p) + 1; /* devices names + max log parameters */ args = dm_malloc((num_devs + 8) * sizeof(char *)); - if (!args || dm_split_words(p, num_devs + 8, 0, args) < num_devs) + if (!args || dm_split_words(p, num_devs + 8, 0, args) < num_devs + 8) goto out_parse; dev_status_str = args[2 + num_devs]; |