[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#853106: lightdm: dm-tool segfaults when XDG_SEAT_PATH doesn't start with '/'



Control: severity -1 wishlist
Control: tags -1 wontfix
Control: reassign -1 glib2.0

Why wouldn't it? This is like taking out a car's wheel and becoming
surprised it doesn't drive very well.

I can't even think of a scenario why it'd be necessary to omit the
initial slash. This is wrong on so many levels. Please refer to the
documentation of the g_variant_is_object_path() function[1] and the
D-Bus specification[2], which I'm going to quote here:

The path must begin with an ASCII '/' (integer 47) character, and must
consist of elements separated by slash characters.

I'm reassigning it to glib2.0 since it's not a bug in LightDM and
there is nothing to fix unless you're suggesting that LightDM provides
a leading slash and covers up for the user's mistake or even
deliberate choice, which is a really bad practice. The glib folks will
most definitely mark it as wontfix/invalid anyway.

[1] https://docs.gtk.org/glib/type_func.Variant.is_object_path.html#description
[2] https://dbus.freedesktop.org/doc/dbus-specification.html

> Changing XDG_SEAT_PATH=org... to XDG_SEAT_PATH=/org... fixes the problem.

A nonexistent problem born from your curiosity. ;)


Reply to: