[Bug 730830] New: hlsdemux: Remove \" sign from decryption KEY url does not always work

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=730830
  GStreamer | gst-plugins-bad | 1.3.2

           Summary: hlsdemux: Remove \" sign from decryption KEY url does
                    not always work
    Classification: Platform
           Product: GStreamer
           Version: 1.3.2
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: memento1987 at interia.pl
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=277304)
 View: https://bugzilla.gnome.org/attachment.cgi?id=277304
 Review: https://bugzilla.gnome.org/review?bug=730830&attachment=277304

Proposition of solution of this bug

When we are parsing decryptin key for HLS stream from manifest file parsing
line has syntax like below one:

#EXT-X-KEY:METHOD=AES-128,URI="https://priv.example.com/key.php?r=52"

Implementation of parsing such line in current version of hlsdemux works in
most situations. It removes \" signes form key URI and returns key as http or
https URI (in current situation it should return:
https://priv.example.com/key.php?r=52

However, I met manifest files which was not parsed properly because of white
characters (or some other characters) at the end of above line. In such
situations I received URL key still containing \" character at the end (in
current situation it could be: https://priv.example.com/key.php?r=52" 
Notice \" sign still being at the end of URL after parsing 

I think it is bug. therefore I propose solution in the attached patch which
worked for me in the above situations and provided me proper key URL for all
manifest files which I tested.

SOLUTION: my proposition of solution is in the attached file

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.