[REPOSTED PATCH] Fix "Incomplete read of frame image da

[genanr]

This patch adds a loop when reading from the frame image pipe. I was getting occasional "Incomplete" messages (in which case, the frame is dropped), even though the whole frame was sent to the pipe. The problem is that all the data was not available in the pipe for zmf when it read from the pipe. Adding the loop allows it to read the entire frame.

The patch was incorrectly posted the first time. I have since posted it to the wiki page here:

http://www.zoneminder.com/wiki/index.php/1.24.2_Patches

The patch seems to be working well for me, I hope it will help others too.

Andy

[Cgroesser]

This patch looks good if its working. I like to use it but, because I get a lot of these errors, i dont no how to patch the original zmf.cpp file. Never worked with C and i'm not shure what the ! chars and so on mean.

[curtishall]

This patch looks good if its working. I like to use it but, because I get a lot of these errors, i dont no how to patch the original zmf.cpp file. Never worked with C and i'm not shure what the ! chars and so on mean.

!= is "does not equal" or "not equal to"

[Cgroesser]

Oke I got the meaning of the ! chars. But whet I try to patch this on the original i got a warning like this:
patch: **** Premature `---' at line 17; check line numbers at line 4. I think this means there are some lines missing in the patch. Mayby something went wrong when cutting and paste this patch to this forum. Is there somebody who has done this with succes??
Chris

[genanr]

Yeah, the posted patch was wrong. Leaving HTML on messed up the patch, even though it was in a code block. It will be posted once I get the thing to post correctly.

I put the patch on the ZoneMinder Wiki

Get it here:

http://www.zoneminder.com/wiki/index.php/1.24.2_Patches

Andy

[knight-of-ni]

Works like a champ on latest svn, revision 3033.

Thanks genanr!

[netwrkr]

Works like a champ on 1.25.0. Any chance we can get this in future versions?

[knight-of-ni]

I recently upgraded to 1.25.0 and was surprised that this patch was not included in the source. Perhaps the author is not aware. I'll post a link to this thread as a feature request and see if we get a bite.

[knight-of-ni]

Just posted this:
http://www.zoneminder.com/forums/viewto ... =8&t=18324