Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 25556 - Gentoo Mozilla (and MozillaFirebird) launcher script fails to escape spaces
Summary: Gentoo Mozilla (and MozillaFirebird) launcher script fails to escape spaces
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Aron Griffis (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-07-30 00:51 UTC by Zhen Lin
Modified: 2004-03-18 19:37 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Zhen Lin 2003-07-30 00:51:09 UTC
In the mozilla script:

function open_mail() {
    if [ "${ALREADY_RUNNING}" -eq "1" ]; then
      exec $MOZ_CLIENT_PROGRAM 'xfeDoCommand(openInbox)' \
        2>/dev/null >/dev/null
    else
      exec $MOZ_PROGRAM $MOZ_LANG $*
    fi
}
                                                                               
                                                               
function open_compose() {
    if [ "${ALREADY_RUNNING}" -eq "1" ]; then
      exec $MOZ_CLIENT_PROGRAM 'xfeDoCommand(composeMessage)' \
        2>/dev/null >/dev/null
    else
      exec $MOZ_PROGRAM $MOZ_LANG $*
    fi
}

In the above 2 functions, $* expands, unescaping any spaces in the command. "$@"
should be used instead.

In the MozillaFirebird script:
${FIREBIRD_PATH}/MozillaFirebird $@

$@ expands like $* unless it is quoted as "$@".

GEnerally, the way to work around escaping problems is to use HTTP escapes
instead, '%20' instead of ' ', for example.

Reproducible: Always
Steps to Reproduce:
Comment 1 Aron Griffis (RETIRED) gentoo-dev 2004-03-18 19:37:08 UTC
I don't know why this hasn't been fixed before now!  It's an obvious bug, I've fixed it.  The firefox script is very different from the original scripts and I don't believe it suffers from this bug.