missing quotationmarks

Bug #53711 reported by Andreas Nitsche
4
Affects Status Importance Assigned to Milestone
acpi-support (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: acpi-support

Line 10 in the file state-funcs is:

test "`cat $DEVICE/device/power/state`" -eq 0 || return

It should be this:

test "`cat "$DEVICE/device/power/state"`" -eq 0 || return

to solve the "Checking battery state..."-problem.

On some systems booting stops with this messages. The cat command cannot reads the given file because ../device/.. is a symlink.

Revision history for this message
Andreas Nitsche (andreas-nitsche) wrote :

It's more than this quotationmarks:

I've an Interface in my system called ?0. This interface is not recognized right and therefore it goes into the cat command line. I worked around this with a regex:

    for DEVICE in /sys/class/net/[a-zA-Z]* ; do

This is now my line 7 in state-funcs.

It could be a solution for normal use of device names.

Revision history for this message
Elliot Hughes (elliot-hughes) wrote :

Confirming as it should be a trivial fix that could easily be solved.

Changed in acpi-support:
status: Unconfirmed → Confirmed
Revision history for this message
Steve Langasek (vorlon) wrote :

I don't believe the current acpi-support implementation suffers from the problem described. Please reopen with more information if you disagree.

Changed in acpi-support (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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