X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fmatch_value.c;h=599f1cfb4bc6863b8432400b25cf18c7b6f2bc33;hb=ba76b1429d2b11935602aaf85174fa6058325d0f;hp=9f02226bdae67109f94a4139cb2f6b60f26cdb34;hpb=108068f268cad453e5d3b54e288917409db1e467;p=collectd.git diff --git a/src/match_value.c b/src/match_value.c index 9f02226b..599f1cfb 100644 --- a/src/match_value.c +++ b/src/match_value.c @@ -1,22 +1,27 @@ /** * collectd - src/match_value.c - * Copyright (C) 2008 Florian Forster + * Copyright (C) 2008 Florian Forster * - * This program is free software; you can redistribute it and/or modify it - * under the terms of the GNU General Public License as published by the - * Free Software Foundation; only version 2 of the License is applicable. + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: * - * This program is distributed in the hope that it will be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * General Public License for more details. + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. * - * You should have received a copy of the GNU General Public License along - * with this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER + * DEALINGS IN THE SOFTWARE. * * Authors: - * Florian Forster + * Florian Forster **/ /* @@ -53,9 +58,18 @@ struct mv_match_s */ static void mv_free_match (mv_match_t *m) /* {{{ */ { + size_t i; + if (m == NULL) return; + if (m->data_sources != NULL) + { + for (i = 0; i < m->data_sources_num; ++i) + free(m->data_sources[i]); + free(m->data_sources); + } + free (m); } /* }}} void mv_free_match */ @@ -115,7 +129,7 @@ static int mv_config_add_data_source (mv_match_t *m, /* {{{ */ /* Allocate space for the char pointers */ new_data_sources_num = m->data_sources_num + ((size_t) ci->values_num); - temp = (char **) realloc (m->data_sources, + temp = realloc (m->data_sources, new_data_sources_num * sizeof (char *)); if (temp == NULL) { @@ -186,13 +200,12 @@ static int mv_create (const oconfig_item_t *ci, void **user_data) /* {{{ */ int status; int i; - m = (mv_match_t *) malloc (sizeof (*m)); + m = calloc (1, sizeof (*m)); if (m == NULL) { - ERROR ("mv_create: malloc failed."); + ERROR ("mv_create: calloc failed."); return (-ENOMEM); } - memset (m, 0, sizeof (*m)); m->min = NAN; m->max = NAN; @@ -263,7 +276,7 @@ static int mv_match (const data_set_t *ds, const value_list_t *vl, /* {{{ */ mv_match_t *m; gauge_t *values; int status; - int i; + size_t i; if ((user_data == NULL) || (*user_data == NULL)) return (-1); @@ -322,7 +335,7 @@ static int mv_match (const data_set_t *ds, const value_list_t *vl, /* {{{ */ if (m->satisfy == SATISFY_ANY) break; } - else if (value_matches == 0) + else { status = FC_MATCH_NO_MATCH; if (m->satisfy == SATISFY_ALL)