On Saturday 08 March 2008 05:11:50 Andy Walls wrote:
> In cx18_stream_alloc(), s->mdl_offset was being used
> uninitialized/memset to 0 in the call to cx18_stream_alloc(), for
> every stream, to do a bounds check on accumulated allocations. This
> appears to be incorrect.
>
> I believe the intent was to use the current running total in
> cx->mdl_offset to perform the bounds check.
>
> Please inspect the attached patch to make sure it makes sense. I'm
> not 100% sure of my current understanding of the mdl_offset
> variables.
>
> Regards,
> Andy
Hi Andy,
Thanks, committed. It was indeed a wrong check and your patch fixed it.
Under normal circumstances this bug will not cause any problems, only
if you try to allocate a really large number of buffers.
Regards,
Hans
_______________________________________________
ivtv-devel mailing list
[email protected]
http://ivtvdriver.org/mailman/listinfo/ivtv-devel
