Quoting error in initrd scripts/local

Bug #115807 reported by [cc]Smart
2
Affects Status Importance Assigned to Milestone
initramfs-tools (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

There's a test lacking proper quoting and thus faulting if the parameter wasn't specified. It should read:

        if [ "${readonly}" = "y" ]; then

Revision history for this message
didier (did447-deactivatedaccount) wrote :

But ${readonly} is always exported in init

On the other hand ${quit} which is also exported is always quoted.

Revision history for this message
Brian Murray (brian-murray) wrote : Re: [Bug 115807] Quoting error in initrd scripts/local

Thanks for taking the time to report this bug and helping to make Ubuntu
better. Which particular script did you notice this bug in? Thanks in
advance.

 status needsinfo
 assignee <email address hidden>

Changed in initramfs-tools:
assignee: nobody → brian-murray
status: Confirmed → Needs Info
Revision history for this message
[cc]Smart (ccsmart) wrote :

Hi.

Subject :)

Inside initrd, file /scripts/local

Greetings.

On Mon, 2007-05-21 at 16:05 +0000, Brian Murray wrote:
> Thanks for taking the time to report this bug and helping to make Ubuntu
> better. Which particular script did you notice this bug in? Thanks in
> advance.
>
> status needsinfo
> assignee <email address hidden>
>
>
> ** Changed in: initramfs-tools (Ubuntu)
> Assignee: (unassigned) => Brian Murray
> Status: Confirmed => Needs Info
>

Changed in initramfs-tools:
assignee: brian-murray → nobody
importance: Undecided → Low
status: Incomplete → Triaged
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (17.3 KiB)

This bug was fixed in the package initramfs-tools - 0.92bubuntu1

---------------
initramfs-tools (0.92bubuntu1) intrepid; urgency=low

  * Merge from debian unstable, remaining changes:
    - Change Build-Depends to Build-Depends-Indep.
    - Use busybox-initramfs for Depends.
    - Do not source udev.conf.
    - Set BUSBOXDIR properly.
    - Keep kinit and gzip in initrd.
    - Use -Qb for for module loading to honor blacklists via the use of
      MODPROBE_OPTIONS.
    - all_generic_ide script.
    - Remove lvm local-top script (conflicts with lvm-common).
    - takeover=1 in update-initramfs.
    - Allow the mounting of a root filesystem as a loop device on top
      of a host filesystem, used for wubi installs.
    - Add vfat support to the initramfs.
    - Reduce timeout to 30 seconds -- corner cases (giant
      disk arrays, clusters) will need to provide their own rootdelay=
      boot arg.
    - Keep more meaningful text for no root device panic.
    - Provide a clearer error on mount failure of the Windows host
      filesystem.
    - Add 2>&1 to the run-init line.
    - Source /scripts/functions in resume script.
    - resume UUID upgrade.
    - Add mountroot failure support, to allow meaningful messages when
      no root device can be found.
    - Panic if either the root device doesn't exist, or vol_id cannot
      identify it. Allows for mountroot failure hooks for md devices
      to be displayed.
    - Ignore blacklist for forced vga= usage.
    - Loop for 5 seconds waiting for $suspend to show up, adjustable
      with the resumedelay= command-line arg.
    - Load virtio_pci for MODULES=most.
    - Usplash pulsates until the real init takes over.
    - Drop scripts/init-premount/thermal as force_load can do this
      for us during initramfs creation.
    - Load thermal modules on lpia.
    - Allow for alternative PREREQS to be specified using | as the
      separator.
    - Add udf to auto_add_modules. It was originally added to
      dep_add_modules, however dep_add_modules now auto-detects the
      modules needed for the root filesystem.
  * hook-functions:
    - Refer to /lib/firmware/${version} where version is the kernel
      version we are building an initramfs for.
    - Do not copy /lib/udev/firmware.agent to the initramfs. Ubuntu's
      udev has /lib/udev/firmware_helper instead, and the udev hook
      copies this to the initramfs for us.
    - Dropped the manual_add_firmware function, as initramfs-tools now
      checks the firmware field in the module it is copying, and copies
      the firmware required by that module if its available.
  * Drop hooks/keymap and the KEYMAP variable setting in
    conf/initramfs.conf as console-setup takes care of keymap loading.
  * hooks/udevhelper: Drop, udev already copies what it needs.
  * hooks/legacylvm: Drop, as it refers to binaries and directories our
    lvm package doesn't have.

initramfs-tools (0.92b) unstable; urgency=low

  [ maximilian attems ]
  * update-initramfs: mbr_check() fix for /dev/md/X naming. (closes: #469312)
    Thanks to Axel Beckert <email address hidden> for report.
  * hook-functions: MODULES=dep fix error message.

  [ Glennie Vignarajah ]
 ...

Changed in initramfs-tools:
status: Triaged → 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.