Code

src/plugin.[ch]: Remove the plugin_dispatch_values_secure() function.
authorFlorian Forster <octo@collectd.org>
Mon, 21 Jan 2013 09:29:30 +0000 (10:29 +0100)
committerFlorian Forster <octo@collectd.org>
Mon, 21 Jan 2013 09:29:30 +0000 (10:29 +0100)
Since values are now *always* copied, we don't need the secure version
anymore.

src/aggregation.c
src/network.c
src/nfs.c
src/plugin.c
src/plugin.h

index 96dc337831ffe08063cd938ba604e6ca3260d4ae..0c0f19d6fe1585301443754bf6bc8933511d8bc5 100644 (file)
@@ -367,7 +367,7 @@ static int agg_instance_read_func (agg_instance_t *inst, /* {{{ */
   vl->values = &v;
   vl->values_len = 1;
 
-  plugin_dispatch_values_secure (vl);
+  plugin_dispatch_values (vl);
 
   vl->values = NULL;
   vl->values_len = 0;
index 72e3484ecaa1216f8e35374e2903355412c31fad..8a5bc6af67a02ce4770a1af14bc82433cc507610 100644 (file)
@@ -460,7 +460,7 @@ static int network_dispatch_values (value_list_t *vl, /* {{{ */
     }
   }
 
-  plugin_dispatch_values_secure (vl);
+  plugin_dispatch_values (vl);
   stats_values_dispatched++;
 
   meta_data_destroy (vl->meta);
@@ -3318,13 +3318,13 @@ static int network_stats_read (void) /* {{{ */
        vl.values[0].derive = (derive_t) copy_octets_rx;
        vl.values[1].derive = (derive_t) copy_octets_tx;
        sstrncpy (vl.type, "if_octets", sizeof (vl.type));
-       plugin_dispatch_values_secure (&vl);
+       plugin_dispatch_values (&vl);
 
        /* Packets received / send */
        vl.values[0].derive = (derive_t) copy_packets_rx;
        vl.values[1].derive = (derive_t) copy_packets_tx;
        sstrncpy (vl.type, "if_packets", sizeof (vl.type));
-       plugin_dispatch_values_secure (&vl);
+       plugin_dispatch_values (&vl);
 
        /* Values (not) dispatched and (not) send */
        sstrncpy (vl.type, "total_values", sizeof (vl.type));
@@ -3333,28 +3333,28 @@ static int network_stats_read (void) /* {{{ */
        vl.values[0].derive = (derive_t) copy_values_dispatched;
        sstrncpy (vl.type_instance, "dispatch-accepted",
                        sizeof (vl.type_instance));
-       plugin_dispatch_values_secure (&vl);
+       plugin_dispatch_values (&vl);
 
        vl.values[0].derive = (derive_t) copy_values_not_dispatched;
        sstrncpy (vl.type_instance, "dispatch-rejected",
                        sizeof (vl.type_instance));
-       plugin_dispatch_values_secure (&vl);
+       plugin_dispatch_values (&vl);
 
        vl.values[0].derive = (derive_t) copy_values_sent;
        sstrncpy (vl.type_instance, "send-accepted",
                        sizeof (vl.type_instance));
-       plugin_dispatch_values_secure (&vl);
+       plugin_dispatch_values (&vl);
 
        vl.values[0].derive = (derive_t) copy_values_not_sent;
        sstrncpy (vl.type_instance, "send-rejected",
                        sizeof (vl.type_instance));
-       plugin_dispatch_values_secure (&vl);
+       plugin_dispatch_values (&vl);
 
        /* Receive queue length */
        vl.values[0].gauge = (gauge_t) copy_receive_list_length;
        sstrncpy (vl.type, "queue_length", sizeof (vl.type));
        vl.type_instance[0] = 0;
-       plugin_dispatch_values_secure (&vl);
+       plugin_dispatch_values (&vl);
 
        return (0);
 } /* }}} int network_stats_read */
index 461e806f90ab76d0ed97747043ca1a1430a6f12c..5c6f620781c59b4c9b284a2f3808a858fa8d4b1f 100644 (file)
--- a/src/nfs.c
+++ b/src/nfs.c
@@ -246,7 +246,7 @@ static void nfs_procedures_submit (const char *plugin_instance,
                vl.values = values + i;
                sstrncpy (vl.type_instance, type_instances[i],
                                sizeof (vl.type_instance));
-               plugin_dispatch_values_secure (&vl);
+               plugin_dispatch_values (&vl);
        }
 } /* void nfs_procedures_submit */
 
index 0aa33b1dce924e4d88329d50c68c0cf400f1265c..547b5eb56da30f590cfbc3dc74c760d162bf5518 100644 (file)
@@ -1936,11 +1936,6 @@ int plugin_dispatch_values (value_list_t const *vl)
        return (0);
 }
 
-int plugin_dispatch_values_secure (const value_list_t *vl)
-{
-  return (plugin_dispatch_values (vl));
-} /* int plugin_dispatch_values_secure */
-
 int plugin_dispatch_notification (const notification_t *notif)
 {
        llentry_t *le;
index c28709e00a0458b100c0ce1bd68c488ed41ba8c9..635ff308f2cd4f83278554f4f79759a5b2efaeb4 100644 (file)
@@ -327,7 +327,6 @@ int plugin_unregister_notification (const char *name);
  *              function.
  */
 int plugin_dispatch_values (value_list_t const *vl);
-int plugin_dispatch_values_secure (const value_list_t *vl);
 int plugin_dispatch_missing (const value_list_t *vl);
 
 int plugin_dispatch_notification (const notification_t *notif);