Bug#492925: plugin argument parsing should be case insensitive

Petter Reinholdtsen pere at hungry.com
Tue Jul 29 22:59:22 UTC 2008

Package: mozilla-plugin-vlc
Version: 0.8.6.h-1
User: debian-edu at lists.debian.org
Usertag: debian-edu

When trying to play the embedded video available from
<URL: http://www1.nrk.no/nett-tv/klipp/386768 > with the
mozilla-plugin-vlc plugin, no video is started.  This is partly
because the plugin fail to detect any URL to play.  This in turn is
caused by the code looking for plugin arguments.  The arguments on
this page are mixed case, and the code in the plugin only look for
lowercase strings.

The arguments are named 'Url', 'Filename', 'AutoStart', etc.  The
plugin code in mozilla/vlcplugin.cpp have tests like this:

  !strcmp( argn[i], "filename")

I believe these tests should be changed to use strcasecmp instead, ie
like this:

  !strcasecmp( argn[i], "filename")

Changing this in the plugin did not solve the problem with the test
page mentioned above, but got the plugin a step closer as it start to
complain about access problems instead of not trying to play anything.

This test page work with the mplayer and kaffeine plugin, so it should
be possible to get it working with vlc too.  I've collected some
browser multimedia test results on
<URL:http://wiki.debian.org/DebianEdu/BrowserMultimedia> in case you
want to see how the vlc plugin compare to the alternatives in Debian.

Happy hacking,
Petter Reinholdtsen

More information about the pkg-multimedia-maintainers mailing list