Bug#373222: vdr-plugin-dvd: Make dvd reading more fault tolerant

[Tobias Grimm]

schorpp wrote:

> hi, pls dont kill dvd playing just due to one sector reading fault:
> pls apply this patch or tell dvdnav somehow to reread or make sw error-correction:

> -      if (dvdnav_get_next_cache_block(nav, &cache_ptr, &event, &len) != DVDNAV_STATUS_OK) {
> +	do {
> +		r = dvdnav_get_next_cache_block(nav, &cache_ptr, &event, &len);
> +      		if (r == DVDNAV_STATUS_OK) break;
> +		l --;
> +	} while ( l > 0);
> +
> +	if (r != DVDNAV_STATUS_OK) {
>            Skins.Message(mtError, tr("Error.DVD$Error fetching data from DVD!"));

This bug is lying around for quite a while now. I never got an answer from
 upstream about this issue. (The dvd plugin development seems to be more
ore less dead at the moment.)

I tried to find a reference implementation that uses dvdnav_get_next* in a
similar way to work around read errors, but couldn't find one. The common
usage of dvdnav_get_next* seems to be, to abort playback when an error occurs.

Tobias


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 260 bytes
Desc: OpenPGP digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-vdr-dvb-devel/attachments/20081129/112bb109/attachment.pgp