Video resume for Samsung TV

Han Holl
2013-05-17
2014-03-10
  • Han Holl
    Han Holl
    2013-05-17

    Hi,

    I was slightly put off that video resume did not work with my Samsung series 6 tv.
    I started looking at the source and the debugging output, and noticed that

    1. the tv sends an X_SetBookmark command on exit from a video

    2. minidlna correctly stores this in the BOOKMARKS tables in the database

    Surely, we were almost there.
    The reason minidlna does not send this offset to the tv, is that it expects " sec:dcmInfo" in the filter expression of the browse command. My tv (and surely many  others) sends "<Filter>*</Filter>".

    The following crude patch makes me happy (for the time being), but I can imagine a more elegant solution.

    --- upnpsoap.c.orig     2012-07-10 23:15:16.000000000 +0200
    +++ upnpsoap.c  2013-05-17 10:03:59.666831816 +0200
    @@ -395,9 +395,15 @@
            char *item, *saveptr = NULL;
            u_int32_t flags = 0;
    
    -       if( !filter || (strlen(filter) <= 1) )
    -               /* Not the full 32 bits.  Skip vendor-specific stuff by default. */
    -               return 0xFFFFFF;
    +       if( !filter || (strlen(filter) <= 1) ) {
    +               if ( strcmp(filter, "*") == 0 && h->reqflags & FLAG_SAMSUNG ) {
    +                       return 0xFFFFFFFF;  /* We want FILTER_SEC_DCM_INFO */
    +               } else {
    +                       /* Not the full 32 bits.  Skip vendor-specific stuff by default. */
    +                       return 0xFFFFFF;
    +               }
    +       }
    +
            if( h->reqflags & FLAG_SAMSUNG )
                    flags |= FILTER_DLNA_NAMESPACE;
            item = strtok_r(filter, ",", &saveptr);
    

    Cheers,

    Han Holl

     
  • Han Holl
    Han Holl
    2013-05-17

    Something went wrong with the markup there.
    Isn't there a preview possibility? I didn't see one.
    Also the window in which to type is awkwardly tiny.

    I forgot to mention the my patch is against the sources of  Fedora minidlna-1.0.25-2.fc17.x86_64, but with a fuzz factor 3 it also applies to the uptodate git clone.

    Cheers,

    Han Holl

     
  • HPP8140
    HPP8140
    2014-03-06

    Hi,

    Any updates? Hard to read. This does not seem to work on samsung 2013 F series.

    Thanks.

     
  • Justin Maggard
    Justin Maggard
    2014-03-10

    This should work in version 1.1.2.