November 2011 - DVDnav-discuss - lists.mplayerhq.hu

[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] 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] Make sure we get nav packets for all cells in multi angle
by Joakim Plate 09 Dec '12

09 Dec '12

Hi, This is a resend of an old patch that we still keep in xbmc's version of libdvdnav. The problem is that libdvdnav jumps over NAV packets when playing multi angle features. I don't remember all the exact details on this patch anymore since it was many years since i tracked it down. But it is very obvious in xbmc since we use nav packet end/start times to correct timestamps. Without the patch we git gaps in nav packets that doesn's exist in the streams. I think i still have a dvd sample that shows the issue. /Joakim

3 5

libdvdnav 4.1.4
by Jean-Jacques Sarton 09 Dec '12

09 Dec '12

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hello, On Fedora 14 totem and vlc end both with folowing message: libdvdread: Found 1 VTS's libdvdread: Elapsed time 0 libdvdnav: Language 'en' not found, using '' instead libdvdnav: Menu Languages available: libdvdnav: Language 'en' not found, using '' instead libdvdnav: Menu Languages available: libdvdnav: Language 'en' not found, using '' instead libdvdnav: Menu Languages available: libdvdnav: *** pgci_ut handle is NULL *** totem: /builddir/build/BUILD/libdvdnav-4.1.4/src/vm/vm.c:1772: get_ID: Assertion `pgcit != ((void *)0)' failed. Of course tje line totem:... is to be replaced withz vlc:... according to the used player The dvd contain no menu and contain a privat video. If I rename libdvdnav totem/vlc works. Regards, Jean-Jacques Sarton -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iEYEARECAAYFAk0h3bAACgkQM9JbiR3CwQsEEgCggN7qJrKQXspNYBJbCGRh237R +joAn1pQaRWHFN+nASDqRbL6Jgj3CM5S =iNgb -----END PGP SIGNATURE-----

4 6

[PATCH] consuming all memory when scanning dvd "Up"
by John Stebbins 09 Dec '12

09 Dec '12

First a disclaimer. This is a big invasive patch. If you don't like it, my feelings won't be hurt. I will be using it in HandBrake. So if you see problems with it, please let me know. Many dvd's are showing up that have titles that have numerous repeated language unit tables and pgc's. The new release "Up" is an example. I believe "Dark Knight" also exhibits this. When scanning such discs with lsdvd (and other programs that scan all titles), libdvdread will consume many GB of data for these repeated elements and gets very slow (e.g hours to read all titles) on OS X due to reading the duplicate data from uncached raw devices. This patch detects duplicates and reference counts pgc's and pgcit's. When a duplicate is detected, a reference count is incremented instead of allocating new memory and re-reading the data. I opted for reference counting instead of simply returning a failure on duplicate detection because it is perfectly valid to have a title with such duplicates. I could easily imagine titles adding duplicates, then navigating around them.

3 5

[PATCH] Link directly with libdvdcss
by Rafaël Carré 30 Nov '11

30 Nov '11

Hi, I'm sending a patch we've been using in VLC for some years on our mingw32 and darwin binaries. It adds --enable-libdvdcss (off by default) to link directly to dvdcss instead of using dlopen() --- dvdread/misc/dvdread.pc.in.orig 2011-11-25 11:13:16.573585109 -0500 +++ dvdread/misc/dvdread.pc.in 2011-11-25 11:01:31.390088283 -0500 @@ -8,4 +8,4 @@ Version: @VERSION@ Cflags: -I${includedir} -Libs: -L${libdir} -ldvdread +Libs: -L${libdir} -ldvdread @CSS_LIBS@ --- dvdread/src/Makefile.am.orig 2011-11-25 11:20:16.287666356 -0500 +++ dvdread/src/Makefile.am 2011-11-25 11:20:24.147705328 -0500 @@ -10,7 +10,7 @@ dvd_input.c dvd_udf.c md5.c nav_print.c ifo_print.c bitreader.c \ bswap.h dvd_input.h dvdread_internal.h dvd_udf.h md5.h bitreader.h -libdvdread_la_LIBADD = $(DYNAMIC_LD_LIBS) +libdvdread_la_LIBADD = $(DYNAMIC_LD_LIBS) @CSS_LIBS@ libdvdread_la_LDFLAGS = -version-info $(DVDREAD_LT_CURRENT):$(DVDREAD_LT_REVISION):$(DVDREAD_LT_AGE) \ -export-symbols-regex "(^dvd.*|^nav.*|^ifo.*|^DVD.*|^UDF.*)" --- dvdread/configure.ac.orig 2011-11-25 10:59:00.953342325 -0500 +++ dvdread/configure.ac 2011-11-25 11:21:26.732015667 -0500 @@ -137,6 +137,11 @@ dnl --------------------------------------------- dnl dynamic linker dnl --------------------------------------------- +AC_ARG_ENABLE([libdvdcss], [ --enable-libdvdcss force linking against libdvdcss]) + +CSS_LIBS="" +if test x"$enable_libdvdcss" != xyes; then +dnl dlopen libdvdcss case $host in *mingw32*) CFLAGS="-idirafter \$(top_srcdir)/msvc/include $CFLAGS" @@ -157,6 +162,18 @@ AC_SUBST(DYNAMIC_LD_LIBS) ;; esac +else +dnl link with libdvdcss + CSS_LIBS=-ldvdcss + case "${host}" in + *darwin*) CSS_LIBS="${CSS_LIBS} -Wl,-framework,CoreFoundation -Wl,-framework,IOKit" + ;; + esac + AC_CHECK_LIB(dvdcss, dvdcss_interface_2,, AC_MSG_ERROR(You need libdvdcss.so.2 or later)) + AC_CHECK_HEADERS(dvdcss/dvdcss.h,, AC_MSG_ERROR(You need libdvdcss (dvdcss.h))) +fi + +AC_SUBST(CSS_LIBS) dnl --------------------------------------------- dnl cflags -- Rafaël Carré

4 4

remap_loadmap purpose
by Roger Pack 30 Nov '11

30 Nov '11

I always get this message and was wondering if it's possible to quiet it a bit: libdvdnav: Unable to find map file 'C:\Users\packrd/.dvdnav/.map' What is the purpose of remap_loadmap? (the .h file doesn't seem to tell much) Thanks. -roger-

2 2