Просмотр исходного кода

cmapctl: Check return code of cmap_set_uint32

... and report error if it isn't possible to set the key, meaning it's
not possible to trigger clear stats operation.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>
Reviewed-by: Christine Caulfield <ccaulfie@redhat.com>
Jan Friesse 5 месяцев назад
Родитель
Сommit
24ada50949
1 измененных файлов с 8 добавлено и 1 удалено
  1. 8 1
      tools/corosync-cmapctl.c

+ 8 - 1
tools/corosync-cmapctl.c

@@ -802,9 +802,16 @@ static void read_in_config_file(cmap_handle_t handle, char * filename)
 static void clear_stats(cmap_handle_t handle, char *clear_opt)
 {
 	char key_name[CMAP_KEYNAME_MAXLEN + 1];
+	cs_error_t err;
 
 	sprintf(key_name, "stats.clear.%s", clear_opt);
-	cmap_set_uint32(handle, key_name, 1);
+
+	if ((err = cmap_set_uint32(handle, key_name, 1)) != CS_OK) {
+		fprintf(stderr, "Can't set cmap stats clear key %s. Error %s\n",
+		    key_name, cs_strerror(err));
+
+		exit (EXIT_FAILURE);
+	}
 }
 
 int main(int argc, char *argv[])