Re: [PATCH v1 1/1] virtio-snd: add max size bounds check in input cb

[Manos Pitsidianakis]

Thanks for the review Philippe,

On Mon, 08 Jul 2024 11:28, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
> > +            max_size = iov_size(buffer->elem->in_sg, 
> >               buffer->elem->in_num);
> >               for (;;) {
> > +                if (buffer->size >= max_size) {
> > +                    return_rx_buffer(stream, buffer);
>
> return_rx_buffer() could be renamed
> flush_input_stream_to_buffer() for clarity.

return_rx_buffer() is vague but I think flush_input_stream_to_buffer() 
is more vague 🤔. I tried to use short names as much as possible and 
document the purpose in doc comments. Unfortunately the device's state 
machine is complex so the code complexity is necessary :/

Manos