make sure dvdcss_version always has something in it. Even if it isn't found.

[Erik Hovland <erik@…>]

The dvdcss_version string is extracted from the shared library. It might fail if the symbol is not found. That is fine. But in the code:

  if(dvdcss_library != NULL) {
    /*
    char *psz_method = getenv( "DVDCSS_METHOD" );
    char *psz_verbose = getenv( "DVDCSS_VERBOSE" );
    fprintf(stderr, "DVDCSS_METHOD %s\n", psz_method);
    fprintf(stderr, "DVDCSS_VERBOSE %s\n", psz_verbose);
    */
    fprintf(stderr, "libdvdread: Using libdvdcss version %s for DVD access\n",
            *dvdcss_version);

The dvdcss_version string might point to null and then printed out. Which wouldn't crash. But would be garbage.

[Erik Hovland <erik@…>]

make sure dvdcss_version is set to something.

[Nigel]

Close, but dvdcss_version is a pointer to a char pointer. I think something like this is needed:

% svn diff libs/libmythdvdnav/dvd_input.c
Index: libs/libmythdvdnav/dvd_input.c
===================================================================
--- libs/libmythdvdnav/dvd_input.c      (revision 16344)
+++ libs/libmythdvdnav/dvd_input.c      (working copy)
@@ -343,7 +343,7 @@
     fprintf(stderr, "DVDCSS_VERBOSE %s\n", psz_verbose);
     */
     fprintf(stderr, "libdvdread: Using libdvdcss version %s for DVD access\n",
-           *dvdcss_version);
+           dvdcss_version ? *dvdcss_version : "unknown");
     
     /* libdvdcss wrapper functions */
     dvdinput_open  = css_open;

[Erik Hovland <erik@…>]

I like your fix better.

[Nigel]

(In [16360]) Check for bad libdvdcss version. Based on patch by Erik Hovland. Closes #4838