September 2012 - DVDnav-discuss - lists.mplayerhq.hu

[patch] check pkg-config before xxx-config scripts
by Mike Frysinger 04 Dec '13

04 Dec '13

it's easier to manage pkg-config when cross-compiling, and libdvdread installs .pc files, so check those first -mike --- configure.ac (revision 1243) +++ configure.ac (working copy) @@ -89,6 +89,7 @@ AC_PROG_CC AC_PROG_MAKE_SET AC_PROG_INSTALL AC_PROG_LN_S +PKG_PROG_PKG_CONFIG dnl -------------------------------------------------------------- dnl Libtool @@ -185,15 +186,26 @@ AC_ARG_WITH([dvdread-config], [AS_HELP_STRING([--with-dvdread-config=PROG], [dvdread-config program to use @<:@default=from PATH@:>@])], [DVDREAD_CONFIG="$withval"], - [dnl User didn't specify program, search PATH - AC_PATH_PROG([DVDREAD_CONFIG], [dvdread-config], [no]) - test "x$DVDREAD_CONFIG" = xno && \ - AC_MSG_ERROR([dvdread-config required to link with libdvdread]) - ]) -DVDREAD_CFLAGS=`$DVDREAD_CONFIG --cflags` || \ - AC_MSG_ERROR([Could not get libdvdread CFLAGS from $DVDREAD_CONFIG]) -DVDREAD_LIBS=`$DVDREAD_CONFIG --libs` || \ - AC_MSG_ERROR([Could not get libdvdread LIBS from $DVDREAD_CONFIG]) + [DVDREAD_CONFIG=""]) + +dnl by default, search pkg-config, and then fall back to dvdread-config +DVDREAD_PKG_CONFIG="no" +if test "x$DVDREAD_CONFIG" = "x"; then + PKG_CHECK_MODULES([DVDREAD], [dvdread], + [DVDREAD_PKG_CONFIG="yes"], + [dnl User didn't specify program, search PATH + AC_PATH_PROG([DVDREAD_CONFIG], [dvdread-config], [no]) + test "x$DVDREAD_CONFIG" = xno && \ + AC_MSG_ERROR([dvdread-config required to link with libdvdread]) + ]) +fi +if test "x$DVDREAD_PKG_CONFIG" != "xyes"; then + DVDREAD_CFLAGS=`$DVDREAD_CONFIG --cflags` || \ + AC_MSG_ERROR([Could not get libdvdread CFLAGS from $DVDREAD_CONFIG]) + DVDREAD_LIBS=`$DVDREAD_CONFIG --libs` || \ + AC_MSG_ERROR([Could not get libdvdread LIBS from $DVDREAD_CONFIG]) +fi + AC_SUBST([DVDREAD_CFLAGS]) AC_SUBST([DVDREAD_LIBS])

3 5

dvdnav SEGFAULT patch for broken DVD
by Vincent Mussard 12 Nov '13

12 Nov '13

Dear all Maybe it will be useful for someone. Please find attached a patch I made, gathering patch made by other people (especialy from Fabian Keil in an email from Sat, 14 Jul 2012). I added on it an additionnal check (ptt[i].pgcn-1 >= ifo->vts_pgcit->nr_of_pgci_srp) that helps reading one of my broken ripped DVD. Regards Vincent

3 2

[PATCH] use newer dll name
by Roger Pack 06 Oct '13

06 Oct '13

For some reason on windows the dll name these days is almost always libdvdcss-2.dll instead of libdvdcss.dll (gstreamer, xbmc, etc.) This patch seems to work, and would make it to match, which seems better from my vantage. -roger-

5 7

assertion failure with certain DVD's
by Roger Pack 06 Oct '13

06 Oct '13

Hello all. I get the following assertion playing the "Tangled" DVD with mplayer and an up to date libdvdnav, on windows: Assertion failed: (vm->state).pgc->cell_playback[(vm->state).cellN - 1].block_mode != 0, file libdvdnav/vm/vm.c, line 1141 The weird part about this is that in mac/linux, I get this error message, and no failed assertion, and playing continues: libdvdnav: Invalid angle block I'd be happy to send more information, or a (legal) DVD to keep, for any developer that would like to tackle it. It appears that it happens with a few other (Disney?) DVD's, as well: http://lists.mplayerhq.hu/pipermail/mplayer-users/2008-December/075458.html https://trac.handbrake.fr/ticket/125 https://forum.handbrake.fr/viewtopic.php?f=11&t=14189 So I'm guessing it's a libdvdnav bug. As a note, in VLC on windows, this causes playback to "loop" over - cellN=2 over and over, though it doesn't crash--I'm guessing it just swallows the assertion failure, or the failure isn't enabled with that build. Thanks. -roger-

4 8

[PATCH] fix dvdnav.h comment
by Roger Pack 06 Oct '13

06 Oct '13

I believe dvdnav_time_search *is* implemented, just somewhat inaccurately, so patch attempts to update comment in this regard. Thanks. -roger-

2 1

[patch] Fix SIGSEGV in dvdnav_describe_title_chapters
by Sylvain HENRY 05 Oct '13

05 Oct '13

Hi, Please consider checking and applying the attached patch. It fixes a segfault occuring with some DVDs. It seems to be the same problem as the one mentionned here: https://bugs.launchpad.net/ubuntu/+source/libdvdnav/+bug/934471 In my case, "cellnr" and "endcellnr" variables were set to 0 when the crash occured. Cheers Sylvain PS: I'm not registered on the mailing-list, please answer directly to me if necessary

3 8

Re: [DVDnav-discuss] Error in libdvdnav when reading iso images
by Fabian Keil 05 Oct '13

05 Oct '13

Oncaphillis wrote in http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/2012-May/001722.html: > I used to create DVD iso images via k3b and played them via VLC. > With isos of relatively newly released DVD this stops to work and > VLC fails with the message > > libdvdnav: demux error! 00 00 00 (should be 0x000001) > > Playing directly from /dev/dvd still works > > The distro is Fedora16 > + k3b == 2.0.2 > + libdvdcss == 1.2.11 ( I also tested 1.2.12) > + vlc = 1.1.13 > + libdvdnav 4.1.4. I also tried the current SVN build > > I get the same error when playing with mplayer and > I also tried brasero for iso creation. > > Since direct play works it seems like brasero/k3b have > problems to create a proper iso and the problems sits > somewhere in the libs. The above message can be generated when the ripping program has generously padded regions with unreadable sectors with binary zeros and one of the padded blocks is actually referenced in the ifos. In this case it's not the result of a bug in the libs (assuming you meant libdvdnav or libdvdread) but the result of a limitation in the program that is using them in combination with companies that interpret the relevant standards somewhat liberal. I'm not familiar with k3b or brasero and thus don't know if this is the cause of the issue, but at least it would explain why the real disc works and the copy doesn't. I've never seen the issue with discs that haven't obviously been mastered incorrectly, but I suppose it can also happen if the disc is scratched. In my experience, copies of intentionally incorrectly mastered discs that cause this issue when padding with zeroes, can usually be played correctly with VLC when padding with other blocks from the region that have been read correctly instead. It's not perfect, but can be sufficient to solve the issue. I'm still experimenting with this, though. With some of those copies I can trigger at least four different crashes in libdvdnav 4.2.0's dvdnav_describe_title_chapters(), that aren't triggered when ripping with zero padding which lets VLC rejects the disc right away. Recent VLC versions use this function, but one can work around it by undefining HAVE_DVDNAV_DESCRIBE_TITLE_CHAPTERS. VLC 1.1.3 doesn't seem to use the function yet and thus correctly plays the copies out of the box (I only confirmed the latter). One of these issues has already been fixed after the libdvdnav 4.2.0 release (https://bugs.launchpad.net/ubuntu/+source/libdvdnav/+bug/934471) but as far as I can tell the others still exist, or at least don't seem to be fixed in https://github.com/microe/libdvdnav and I haven't seen them mentioned when crawling the archives either. I attached the patches that seem to prevent these crashes for me. I'd appreciate comments on whether or not the skip conditions are reasonable, especially the one in patch 3. Note that I ran into the issues with copies where I now know that the padding has been done somewhat sloppy. I haven't tested yet if these crashes can be reached by playing the original discs. If not, I suspect that with a fancier padding algorithms the copies might play correctly as well. Fabian

4 4

[PATCH] Sanitize PTT start offsets
by John Stebbins 05 Oct '13

05 Oct '13

If the start offset points past the last_byte (outside the region read) stop reading PTTs and adjust nr_of_srpts. Fixes Transformers 3 -- John GnuPG fingerprint: D0EC B3DB C372 D1F1 0B01 83F0 49F1 D7B2 60D4 D0F7

3 5

[PATCH] "dvdnav_jump_to_sector" as an alternative to "dvdnav_time_search" (REV 5: combined args to structs)
by gnosygnu 29 Jul '13

29 Jul '13

This revision includes the following changes: 1) combined multiple arguments into structures (See dvdnav_pos_data_t, dvdnav_cell_data_t, dvdnav_jump_args_t) 2) changed multi-line statements to remove leading commas 3) removed tabs/trailing whitespace 4) replaced "discontinuity tmap indexes" approach with simpler "use entry sector of current cell" to handle DVDs with initial dummy cells. Thanks to all who have read through the patch and/or offered comments. As far as I know, there are no other known pending changes/requests. I am hoping that this patch is at a near-final state. Please let me know what else should be done to get this patch accepted. Thanks.

3 19

[PATCH] "dvdnav_jump_to_sector" as an alternative to "dvdnav_time_search" (REV 6: add mode argument)
by gnosygnu 16 Dec '12

16 Dec '12

This revision features one minor change. It adds a new "mode" parameter to "dvdnav_jump_to_sector". As per the included comment: /* NOTE: Mode is currently unimplemented. Only 0 should be passed. */ /* 1 and -1 are for future implementation */ /* 0: Default. Jump to a time which may be either <> time_in_pts_ticks */ /* 1: After. Always jump to a time that is > time_in_pts_ticks */ /* -1: Before. Always jump to a time that is < time_in_pts_ticks */ dvdnav_status_t dvdnav_jump_to_sector_by_time(dvdnav_t *this, uint64_t time_in_pts_ticks, int32_t mode) ... The corresponding .h interface was also updated. My apologies for not implementing the new parameter, but as I detailed in another email, the change looks quite significant. As far as I know, there are no other known pending changes/requests. I am hoping that this patch is at a final state. Please let me know what else should be done to get this patch accepted. Thanks.

4 6