From d86ee93801aed64e9eaf16849b7b008e0089b0c9 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 7 Aug 2013 18:14:57 +0200 Subject: filter/Route: don't access PcmBuffer attributes directly Return the pointer given by PcmBuffer::Get() instead of reaching into the PcmBuffer object. --- src/filter/RouteFilterPlugin.cxx | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/filter/RouteFilterPlugin.cxx') diff --git a/src/filter/RouteFilterPlugin.cxx b/src/filter/RouteFilterPlugin.cxx index ceb944822..2c0d60cbe 100644 --- a/src/filter/RouteFilterPlugin.cxx +++ b/src/filter/RouteFilterPlugin.cxx @@ -275,12 +275,12 @@ RouteFilter::FilterPCM(const void *src, size_t src_size, // A moving pointer that always refers to channel 0 in the input, at the currently handled frame const uint8_t *base_source = (const uint8_t *)src; - // A moving pointer that always refers to the currently filled channel of the currently handled frame, in the output - uint8_t *chan_destination; - // Grow our reusable buffer, if needed, and set the moving pointer *dest_size_r = number_of_frames * output_frame_size; - chan_destination = (uint8_t *)output_buffer.Get(*dest_size_r); + void *const result = output_buffer.Get(*dest_size_r); + + // A moving pointer that always refers to the currently filled channel of the currently handled frame, in the output + uint8_t *chan_destination = (uint8_t *)result; // Perform our copy operations, with N input channels and M output channels for (unsigned int s=0; s