diff options
| author |   Paul Atkins <patkins@graphiant.com> | 2021-10-04 16:29:52 +0100 |
|---|---|---|
| committer |   Neale Ranns <neale@graphiant.com> | 2021-11-22 09:30:09 +0000 |
| commit | c2d476b26dead1097f120a2bf35cda904c4f55f9 (patch) | |
| tree | a59372deeef7aa31f017f8e3cffed49b244cc077 /src/plugins/ioam/udp-ping/udp_ping_export.c | |
| parent | 19a5f23b23e4eec54d99f105725ddc4df7421a5c (diff) | |
ipfix-export: refactor params to the callback fns
When a new flow-report is created the caller provides 2 callback functions.
These functions both take a pointer to the exporter, plus a pointer to the
source and dest address. However the pointers to the address are not adding
any value as these are always set to the src/dest addresses of the exporter
(which is already being passed). Remove these parameters and leave the
callback functions to get the addresses out of the exporter.
Type: improvement
Signed-off-by: Paul Atkins <patkins@graphiant.com>
Change-Id: I36dec394f30e85cdca120dd8706b5d90f5e07c48
Diffstat (limited to 'src/plugins/ioam/udp-ping/udp_ping_export.c')
| -rw-r--r-- | src/plugins/ioam/udp-ping/udp_ping_export.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/plugins/ioam/udp-ping/udp_ping_export.c b/src/plugins/ioam/udp-ping/udp_ping_export.c index d8926aa01d4..3dc1436a1a9 100644 --- a/src/plugins/ioam/udp-ping/udp_ping_export.c +++ b/src/plugins/ioam/udp-ping/udp_ping_export.c @@ -24,13 +24,11 @@ static u8 * udp_ping_template_rewrite (ipfix_exporter_t *exp, flow_report_t *fr, - ip4_address_t *collector_address, - ip4_address_t *src_address, u16 collector_port, - ipfix_report_element_t *elts, u32 n_elts, - u32 *stream_index) + u16 collector_port, ipfix_report_element_t *elts, + u32 n_elts, u32 *stream_index) { - return ioam_template_rewrite (exp, fr, collector_address, src_address, - collector_port, elts, n_elts, stream_index); + return ioam_template_rewrite (exp, fr, collector_port, elts, n_elts, + stream_index); } static vlib_frame_t * |