plugin dbi: Added lost check for udb_query_prepare_result() return value.
authorPavel Rochnyack <pavel2000@ngs.ru>
Wed, 11 May 2016 21:23:01 +0000 (03:23 +0600)
committerPavel Rochnyack <pavel2000@ngs.ru>
Thu, 24 May 2018 05:33:18 +0000 (12:33 +0700)
src/dbi.c

index 4ef9f8c..f37d04c 100644 (file)
--- a/src/dbi.c
+++ b/src/dbi.c
@@ -547,12 +547,18 @@ static int cdbi_read_database_query(cdbi_database_t *db, /* {{{ */
     sstrncpy(column_names[i], column_name, DATA_MAX_NAME_LEN);
   } /* }}} for (i = 0; i < column_num; i++) */
 
-  udb_query_prepare_result(
+  status = udb_query_prepare_result(
       q, prep_area, (db->host ? db->host : hostname_g),
       /* plugin = */ (db->plugin_name != NULL) ? db->plugin_name : "dbi",
       db->name, column_names, column_num,
       /* interval = */ (db->interval > 0) ? db->interval : 0);
 
+  if (status != 0)
+  {
+    ERROR("dbi plugin: udb_query_prepare_result failed with status %i.", status);
+    BAIL_OUT (-1);
+  }
+
   /* 0 = error; 1 = success; */
   status = dbi_result_first_row(res); /* {{{ */
   if (status != 1) {