Created attachment 442943 [details]
diff file for my proposed patch

Description of problem:
By default, Openbox generates menus via this file:
/usr/share/lxde/openbox/menu.xml

This file generates some menus on-the-fly by executing this command:
/usr/share/openbox/xdg-menu

This command is a python script that reads content of .desktop files in directory:
/usr/share/applications/

If one .desktop file has some fields with values inside double quotes, like this one :
/usr/share/applications/fedora-system-config-selinux.desktop

here is what's inside this file:
Name[fr]="Gestion de SELinux"

Then the xdg-menu script does not escape the double quotes causing an error in the XML generation, like this:
<item label=""Gestion de SELinux"">

Which is obviously incorrect XML causing openbox to not display the menu (ignored because of an error when parsing XML).

Version-Release number of selected component (if applicable):
openbox-3.4.11.2-1.fc13.x86_64
openbox-libs-3.4.11.2-1.fc13.x86_64

How reproducible:
Always

Steps to Reproduce:
0. init 3
1. create a new user with its empty homedir
2. Xorg &
3. export DISPLAY=':0.0'
4. openbox-session
5. right-clic somewhere on the desktop
6. go to the "applications" sub-menu : it's empty
7. check openbox-session output: it points to the double double-quotes error


Actual results:
The sub-menus generated with xdg-menu script are empty.

Expected results:
The xdg-menu script should take care of existing double-quotes from the .desktop files.

Additional info:
I've corrected this by modifying the line 32:

print ' <item label="%s">' % escape(entry.get_name().replace('"',''))

(see attachment diff file)

Bye for now.