klibido klibidoload can not work with spaces in nzb file name

Bug #246350 reported by Danny van Heumen
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
klibido (Ubuntu)
Fix Released
Low
Harald Sitter
Nominated for Karmic by Danny Daemonic

Bug Description

Binary package hint: klibido

klibido comes with a shell script /usr/bin/klibidoload. klibidoload can be linked to NZB-files in order to load them into klibido by double clicking. (Binding the application to the mime-type.) But the script is missing quotes around the $1 occurrences, so it does not correctly handle nzb file names with spaces.

Description: Ubuntu 8.04.1
Release: 8.04

klibido:
  Installed: (none)
  Candidate: 0.2.5-5ubuntu3
  Version table:
     0.2.5-5ubuntu3 0
        500 http://nl.archive.ubuntu.com hardy/universe Packages

When passing an nzb-file with one or more spaces in the file name to /usr/bin/klibidoload it will do nothing. It should have opened op the open NZB window inside klibido asking the user where to store the downloaded files.

Revision history for this message
Andreas Moog (ampelbein) wrote :

debdiff attached.

klibido (0.2.5-5ubuntu4) intrepid; urgency=low

  * Escape $1 in klibidoload to handle files with Spaces correctly (LP: #246350)

 -- Andreas Moog <email address hidden> Sat, 23 Aug 2008 16:04:16 +0200

Changed in klibido:
status: New → Confirmed
Changed in klibido:
assignee: nobody → apachelogger
importance: Undecided → Low
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package klibido - 0.2.5-5ubuntu4

---------------
klibido (0.2.5-5ubuntu4) intrepid; urgency=low

  [ Andreas Moog ]
  * Escape $1 in klibidoload to handle files with Spaces correctly (LP: #246350)

  [ Harald Sitter ]
  * Change automake build dependency to automake1.9
  * Bump Standards-Version to 3.8.0
  * Typo fix in description kde -> KDE
  * Move Homepage from description to own field
  * Fix Anthony Mercatante's email address
  * In debian rules remove applnk/ instead of just applnk/Utilities/
  * Update watch file to new sf.net redirector to make lintian happy
  * Build against libdb4.6++-dev (LP: #199283)
  * Add final new line to debian/newzleech.desktop

 -- Harald Sitter <email address hidden> Mon, 25 Aug 2008 15:01:11 +0200

Changed in klibido:
status: In Progress → Fix Released
Revision history for this message
Danny Daemonic (dannydaemonic) wrote :

I would reopen this bug report if I could - the fix is not working in Ubuntu 9.10.

I imagine it's not working correctly in other versions of Ubuntu, although it might just be related to changes in Bash 4.0.

I've attached a working version of klibidoload. It uses -n AND quotes to test for empty arguments.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.