On Sep 30, 2007, at 06:04, Christoph Bartelmus wrote:
>> Speaking of the igor code... Upon looking at the mem_failure
>> switch in
>> lirc_igorplugusb.c in greater detail, it flat-out looks wrong. You
>> hit the
>> switch, then only free a tiny portion of memory allocated, and only
>> return -ENOMEM if the mem_failure was case 1. Otherwise, we
>> continue on our
> I don't see the problem. This is a fall-through switch, so it should
> work fine.
Blah. Once again, I wrote too soon w/o carefully re-reading. In my
mind, I inserted break's all over the place in the switch. Sorry
> Based on your v5 patch, I have just put a new version to CVS.
> I fixed some minor glitches in the patch and some bugs in the original
> Please compare your latest version with the current version in CVS.
> Apart from some more fixes in CVS, it should be the same now. If
> anything is still missing, please let me know.
I'll try to give it a look either tonight or tomorrow and see what I
can see (hopefully, nothing!).