February 2014 - DVDnav-discuss - lists.mplayerhq.hu

Can't watch DVD
by jan.maske＠abwesend.de 23 May '14

23 May '14

Hi I bought a new DVD, but can't watch it. Using libdvdread 4.2.0 lsdvd eats a lot of memory and gives the following messages: $ lsdvd libdvdread: Using libdvdcss version 1.2.13 for DVD access *** libdvdread: CHECK_VALUE failed in ifo_read.c:1251 *** *** for vts_ptt_srpt->title[i].ptt[j].pgn != 0 *** *** libdvdread: CHECK_VALUE failed in ifo_read.c:1980 *** *** for pgci_ut->nr_of_lus < 100 *** Please send bug report - no VTS_TMAPT ?? *** libdvdread: CHECK_VALUE failed in ifo_read.c:1251 *** *** for vts_ptt_srpt->title[i].ptt[j].pgn != 0 *** *** libdvdread: CHECK_VALUE failed in ifo_read.c:1980 *** *** for pgci_ut->nr_of_lus < 100 *** Please send bug report - no VTS_TMAPT ?? *** libdvdread: CHECK_VALUE failed in ifo_read.c:1251 *** *** for vts_ptt_srpt->title[i].ptt[j].pgn != 0 *** With a more recent version from git it doesn't use that much memory and outputs just: $ lsdvd *** libdvdread: CHECK_VALUE failed in ifo_read.c:1246 *** *** for vts_ptt_srpt->title[i].ptt[j].pgn != 0 *** *** libdvdread: CHECK_VALUE failed in ifo_read.c:1246 *** *** for vts_ptt_srpt->title[i].ptt[j].pgn != 0 *** libdvdread: Invalid IFO for title 2 (VTS_02_0.BU). Can't open ifo 2! And mplayer produces: Playing dvd://2. There are 99 titles on this DVD. There are 1 angles in this DVD title. *** libdvdread: CHECK_VALUE failed in ifo_read.c:1246 *** *** for vts_ptt_srpt->title[i].ptt[j].pgn != 0 *** *** libdvdread: CHECK_VALUE failed in ifo_read.c:1246 *** *** for vts_ptt_srpt->title[i].ptt[j].pgn != 0 *** libdvdread: Invalid IFO for title 2 (VTS_02_0.BU). Cannot open the IFO file for DVD title 2. No stream found to handle url dvd://2 I tried it on different computers, so it is not a hardware problem. Only one Window Media Player could actually play it. Any help would be appreciated. Jan

5 6

[PATCH] Initialize dvd_reader_t struct in DVDOpenImageFile
by Richard Hulme 02 Mar '14

02 Mar '14

Hi, This is a patch I found in MythTV's copy of libdvdread that seems like it could/should be upstream too. It ensures that the dvd_reader_t struct is initialized (e.g. the css_state field is currently undefined if 'have_css' is false). The patch is part of a bigger MythTV-specific change by Chris Pinkham: https://github.com/MythTV/mythtv/commit/3af3489 Richard Hulme

3 9

[PATCH] os2: include os2safe.h
by KO Myung-Hun 02 Mar '14

02 Mar '14

This header makes some non-high-memory-safe OS/2 APIs high-memory safe. --- src/vm/vm.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/vm/vm.c b/src/vm/vm.c index 192e868..d956a38 100644 --- a/src/vm/vm.c +++ b/src/vm/vm.c @@ -55,6 +55,7 @@ #ifdef __OS2__ #define INCL_DOS +#include <os2safe.h> #include <os2.h> #include <io.h> /* setmode() */ #include <fcntl.h> /* O_BINARY */ -- 1.7.3.2

2 2

libdvdnav so version docs
by Marcel J.E. Mol 02 Mar '14

02 Mar '14

libdvdnav configure.ac domucemnts libtool version-info as current:age:revision. According to libtool documentation this should be current:revision:age. See attached patch to fix this. Thanks, -Marcel -- ======-------- Marcel J.E. Mol MESA Consulting B.V. =======--------- ph. +31-(0)6-54724868 P.O. Box 112 =======--------- marcel(a)mesa.nl 2630 AC Nootdorp __==== www.mesa.nl ---____U_n_i_x______I_n_t_e_r_n_e_t____ The Netherlands ____ They couldn't think of a number, Linux user 1148 -- counter.li.org so they gave me a name! -- Rupert Hine -- www.ruperthine.com

2 2

libdvdnav|read on git.v.o
by Jean-Baptiste Kempf 02 Mar '14

02 Mar '14

Dear dvdnav developers, As you might be aware, I spent a lot of time moving the libdvdnav and libdvdread SVN to git. It took an extremely long time, because I decided to provide correct authorship, and remove the extremely numerous "Patch by" messages with correct credit. A lot of branches, notably from Erik and Diego got merged, and a bit of cleaning has been done. All SVN commits have been merged too. The code was adapted for the upcoming libdvdcss release and the build systems were rewritten to be simpler. http://git.videolan.org/?p=libdvdnav.git;a=summary http://git.videolan.org/?p=libdvdread.git;a=summary Everyone that had commit access here is welcome and can have commit access to this new repository. People like Erik, gnosygnu or others that care about libdvdnav or libdvdread can have commit access too. You just need to mail me a pubkey for ssh/git. I'd like that those projects come again to life, notably because of the new protections like ARccOS, and because the quality is not good enough, and the crashes too present. Some branches have never been committed and should be merged too. The main differences you will see is that now we need git-formatted patches to have proper credit (author != commiter) and that we intend to keep it like this. The second one is the autotools build-system. And the last one is the split of the vm.c file. With my kindest regards, -- Jean-Baptiste Kempf http://www.jbkempf.com/ - +33 672 704 734 Sent from my Electronic Device

4 5

libdvdread so version
by Marcel J.E. Mol 23 Feb '14

23 Feb '14

Hi, When building libdvdread from svn I end up with libdvdread.so.4.1.2. But when building from the new git repository I get libdvdread.so.3.2.1. I'm not to familar with autotools, but configure.ac defines DVDREAD_LT_CURRENT=5 DVDREAD_LT_AGE=1 DVDREAD_LT_REVISION=2 in both versions. It sems somehow LT_AGE an LT_REVISION are reversed??? For what it's worth, I build on Fedora 20 and on openembedded with the same results. A patch that would fix this is attached (somehow git format-patch does nothing...) Thanks, -Marcel -- ======-------- Marcel J.E. Mol MESA Consulting B.V. =======--------- ph. +31-(0)6-54724868 P.O. Box 112 =======--------- marcel(a)mesa.nl 2630 AC Nootdorp __==== www.mesa.nl ---____U_n_i_x______I_n_t_e_r_n_e_t____ The Netherlands ____ They couldn't think of a number, Linux user 1148 -- counter.li.org so they gave me a name! -- Rupert Hine -- www.ruperthine.com

2 2

[PATCH] Fix DVDDiscID from reading non-existent IFO files
by Steve Dibb 18 Feb '14

18 Feb '14

First patch, and from a C noob even, so go easy on me. :) In dvd_reader.c, the DVDDiscID function gets the MD5 from the total contents of the IFO files on the disc. However, it gets it by looping through the the first ten IFO files, whether they exist or not. This results in spewage, where a disc has less than ten. In this case, Dragonheart only has two: libdvdnav:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VTS_03_0.IFO failed libdvdnav:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VTS_04_0.IFO failed libdvdnav:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VTS_05_0.IFO failed libdvdnav:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VTS_06_0.IFO failed libdvdnav:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VTS_07_0.IFO failed libdvdnav:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VTS_08_0.IFO failed libdvdnav:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VTS_09_0.IFO failed Instead of using a set limit of 10 IFOs, this patch counts the number of VTSes existing on the DVD, and will use that number instead if it is less than 10. Also, doesn't break backwards compatability -- MD5 sum is the same before and after the patch. :) --- src/dvd_reader.c.orig 2014-02-16 15:39:51.890964032 -0700 +++ src/dvd_reader.c 2014-02-16 16:37:42.269821221 -0700 @@ -70,6 +70,7 @@ #include "dvd_input.h" #include "dvdread/dvd_reader.h" #include "md5.h" +#include "dvdread/ifo_read.h" #define DEFAULT_UDF_CACHE_LEVEL 1 @@ -1329,16 +1330,31 @@ { struct md5_ctx ctx; int title; + int title_sets; int nr_of_files = 0; + ifo_handle_t *vmg_ifo; /* Check arguments. */ if( dvd == NULL || discid == NULL ) return 0; - /* Go through the first 10 IFO:s, in order, + vmg_ifo = ifoOpen( dvd, 0 ); + if( !vmg_ifo ) { + fprintf( stderr, "libdvdread: DVDDiscId, failed to " + "open VMG IFO!\n" ); + return -1; + } + + title_sets = vmg_ifo->vmgi_mat->vmg_nr_of_title_sets + 1; + ifoClose( vmg_ifo ); + + if( title_sets > 10 ) + title_sets = 10; + + /* Go through the first IFO:s, in order, up until the tenth, * and md5sum them, i.e VIDEO_TS.IFO and VTS_0?_0.IFO */ md5_init_ctx( &ctx ); - for( title = 0; title < 10; title++ ) { + for( title = 0; title < title_sets; title++ ) { dvd_file_t *dvd_file = DVDOpenFile( dvd, title, DVD_READ_INFO_FILE ); if( dvd_file != NULL ) { ssize_t bytes_read;

3 3

[PATCH] Initialise SPRM 15 (audio capabilities) to meaningful value
by Richard Hulme 18 Feb '14

18 Feb '14

Hi, The attached patch initialises the audio capabilities register (SPRM 15) to indicate that all audio features are available. My region 2 version of 'Monsters University' queries this register in 'first play' and jumps to an invalid title if it is zero. Is it worth having a 'dvdnav_set_audio_capabilities' function? This is the first DVD I've come across that is obviously querying the register, but it was only obvious because the disc wouldn't play. I don't know whether there could be instances where indicating that everything is supported could cause problems. Richard.

2 1

Re: [DVDnav-discuss] [SCM] libdvdnav/master: Merge tag 'upstream/4.9.0__gita5c13254'
by Bálint Réczey 17 Feb '14

17 Feb '14

Dear Jean-Baptiste and DVDnav developers, We applaud your efforts in contributing to libdvdnav. Having libdvdnav's code in git and having a new very active developer is a big help to the project. We at Debian would happily use the libdvdnav code from git as the base of our packages, but from the linked thread [1] it seems there is no established consensus regarding the state of the VideoLAN-hosted repository. I would like to ask all of you (developers contributing either to the original svn and the new git repositories) to decide if the repository at git.videolan.org/libdvdnav.git is a continuation of libdvdnav's development or not. This is needed to prevent the fragmentation of development which would unnecessarily complicate life for projects using libdvdnav. I'm somewhat biased towards migrating libdvdnav to git in this step and continuing development in VLC's repo since it is ready and moving to git is probably unavoidable. Either way I would be glad to see a public announcement on dvdnav-discuss clarifying the situation. Thanks, Balint [1] http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/2013-November/002001.html PS: Please CC-m, I'm not on dvdnav-discuss 2014-02-12 14:47 GMT+01:00 Reinhard Tartler <siretart(a)gmail.com>: > FYI: > > I intend to switch our system libdvdnav library to the version > provided on videolan. Some discussion on this fork can be seen on the > dvdnav mailing list: > > http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/2013-November/002001.html > > Please tell me if you have concerns, but I think this is the right > thing to do for Debian. FWIW, I believe that what we currently have in > git/master is good enough for unstable. I'll upload later this week > unless someone objects. > > Best, > Reinhard > > On Mon, Feb 10, 2014 at 10:07 PM, <siretart(a)users.alioth.debian.org> wrote: >> Imported Upstream version 4.9.0~~gita5c13254 >> MIME-Version: 1.0 >> Content-Type: text/plain; charset=UTF-8 >> X-Git-Refname: refs/heads/master >> X-Git-Reftype: branch >> X-Git-Oldrev: 7f97a3df880e5119aa503f3463293024189ce86d >> X-Git-Newrev: c509b40700bb19d935e1d811e857c6beb23b6ce9 >> >> The following commit has been merged in the master branch: >> commit 7ffeb8e3639ed334b970338c513468ba784578a4 >> Merge: 7f97a3df880e5119aa503f3463293024189ce86d 63ef6f4abf8068db62bb2ea4c6dbc09e86056dfd >> Author: Reinhard Tartler <siretart(a)tauware.de> >> Date: Tue Feb 11 02:18:52 2014 +0000 >> >> Merge tag 'upstream/4.9.0__gita5c13254' >> >> Upstream version 4.9.0~~gita5c13254 >> >> Conflicts: >> .gitignore >> >> >> -- >> libdvdnav packaging ... > regards, > Reinhard

4 5

Debugging non-playable movie
by Hans Petter Selasky 16 Feb '14

16 Feb '14

Hi, A friend of mine which does not know so much about computers bought a movie and wanted to play it under Linux. Unfortunately the movie doesn't play, due to some strange formatting, which I think is done by purpose by the movie maker First error observed: ./libdvdread-4.2.1/src/ifo_read.c The ifo_filename needs to be extended one byte, hence the filenames are one byte too long and snprintf's need to be updated too. char ifo_filename[14]; Second error observed: if(vts_ptt_srpt->nr_of_srpts > info_length / sizeof(*data)) { fprintf(stderr, "libdvdread: PTT search table too small %d %d.\n", (int)vts_ptt_srpt->nr_of_srpts, (int)(info_length / sizeof(*data))); goto fail; } vts_ptt_srpt->nr_of_srpts = 99 info_length / sizeof(*data) = 2 After making these changes only the copy warning is playable. I can test patches in libdvdread. Please keep me CC'ed hence I'm not subscribed to this list. How to proceed next? --HPS

2 1