Package: xine-ui
Version: 0.99.6-1
Severity: normal

xine --enqueue splits any filename provided on the commandline if they
have a comma in the filename.

It does this because --enqueue gets transformed in main.c:main() to
allocate a session_argv array with mrl=%s and passes that off to the
session code to interpret in session.c:session_handle_subopt, which
just stupidly blindly splits every string at every "," via getsubopt()
right at the top of session_handle_subopt().  There is no way around
this - you can't even escape commas in the filename with "\" or the
like.  Seems someone forgot that mrls get passed through getsubopt and
can of course be arbitrary strings so no character should be entitled
to act as delimeter.  The mrl case should be handled via some other
means - probably via a separate array since ideally you should be able
to even handle a filename that looks like "token=value".  It should be
easy - only interpret tokens provided via
S=token1,token2,token3,token4=value,token5 etc, and all other options
parsed through getopt(), but any other argument should be interpreted
as a filename.

Continue to support "xine -S mrl=..." for backwards compatibility by
all means, but xine -S ... --enqueue ... ... should be the preferred
method.

-- System Information:
Debian Release: 6.0.5
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable'), (5, 'testing'), (1, 
'experimental'), (1, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.5-trunk-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_AU.UTF-8, LC_CTYPE=en_AU.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages xine-ui depends on:
ii  libc6                  2.13-10           Embedded GNU C Library: Shared lib
ii  libcurl3-gnutls        7.25.0-1          easy-to-use client-side URL transf
ii  liblircclient0         0.8.3-5           infra-red remote control support -
ii  libpng12-0             1.2.44-1+squeeze4 PNG library - runtime
ii  libreadline6           6.1-3             GNU readline and history libraries
ii  libx11-6               2:1.4.99.901-2    X11 client-side library
ii  libxext6               2:1.3.0-3         X11 miscellaneous extension librar
ii  libxft2                2.1.14-2          FreeType-based font drawing librar
ii  libxine1               1:1.1.20-0.1      Xine video/media player library, m
ii  libxine1-ffmpeg        1:1.1.20-0.1      MPEG-related plugins for libxine1
ii  libxine1-x             1:1.1.20-0.1      X desktop video output plugins for
ii  libxinerama1           2:1.1.1-3         X11 Xinerama extension library
ii  libxtst6               2:1.1.0-3         X11 Testing -- Record extension li
ii  libxv1                 2:1.0.5-1         X11 Video extension library
ii  libxxf86vm1            1:1.1.0-2         X11 XFree86 video mode extension l

Versions of packages xine-ui recommends:
ii  xdg-utils            1.0.2+cvs20100307-2 desktop integration utilities from

xine-ui suggests no packages.

-- no debconf information

-- debsums errors found:
debsums: changed file /var/lib/xine/xine.desktop (from xine-ui package)


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to