Debian Bug report logs - #703746
live-boot: CIFS-backed netboot fails with segfault of mount.cifs

version graph

Package: src:live-boot; Maintainer for src:live-boot is Live Systems Maintainers <debian-live@lists.debian.org>;

Reported by: Andrew Jeffery <andrew.jeffery@ultra-avalon.com>

Date: Sat, 23 Mar 2013 00:42:02 UTC

Severity: normal

Tags: patch

Found in version live-boot/3.0.1-1

Fixed in version live-boot/4.0~a11-1

Done: Daniel Baumann <daniel.baumann@progress-technologies.net>

Reply or subscribe to this bug.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to debian-bugs-dist@lists.debian.org, Debian Live Project <debian-live@lists.debian.org>:
Bug#703746; Package src:live-boot. (Sat, 23 Mar 2013 00:42:06 GMT) Full text and rfc822 format available.

Acknowledgement sent to Andrew Jeffery <andrew.jeffery@ultra-avalon.com>:
New Bug report received and forwarded. Copy sent to Debian Live Project <debian-live@lists.debian.org>. (Sat, 23 Mar 2013 00:42:06 GMT) Full text and rfc822 format available.

Message #5 received at submit@bugs.debian.org (full text, mbox):

From: Andrew Jeffery <andrew.jeffery@ultra-avalon.com>
To: Debian Bug Tracking System <submit@bugs.debian.org>
Subject: live-boot: CIFS-backed netboot fails with segfault of mount.cifs
Date: Sat, 23 Mar 2013 11:09:45 +1030
[Message part 1 (text/plain, inline)]
Source: live-boot
Version: 3.0.1-1
Severity: normal
Tags: patch

I'm trying to netboot an x86 system which has multiple NICs, though only
one is recognised by the BIOS/PXE boot firmware and as such the full
netboot process should happen over this interface. The real root image
is hosted on a CIFS share but live-boot fails to mount the share during
early userspace.

The trouble is caused by a segfault of mount.cifs, which I must say is
difficult to debug in early userspace. The mount attempt only seemed to
fail in the live-boot scripts; attempting to mount the filesystem from
the recovery shell provided in early-userspace succeeded. Investigating,
it seems that the mount options as provided to mount.cifs in the
live-boot scripts are at fault in that the option variable is quoted,
including the -o, making it an unexpected argument to mount.cifs.
Arguably mount.cifs should do something better than simply segfault,
however it's still the case that live-boot is not passing the parameters
correctly.

The attached patch simply doesn't quote CIFSOPTS when passing the value
to mount.cifs. This may not be the best approach; a better one might be
to provide -o separate to the CIFSOPTS value. I'm happy to revise the
patch if that's prefered.

The system information provided below isn't terribly relevant, however
having only ever filed a series of Debian bugs in the last 20 minutes
I'm not sure if I should take it out.

-- System Information:
Debian Release: 7.0
  APT prefers testing
  APT policy: (650, 'testing'), (600, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.7.2 (SMP w/8 CPU cores)
Locale: LANG=en_AU.UTF-8, LC_CTYPE=en_AU.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
[0003-CIFS-Don-t-quote-CIFSOPTS-for-mount.cifs.patch (text/x-diff, attachment)]

Marked as fixed in versions live-boot/4.0~a11-1. Request was from Daniel Baumann <daniel.baumann@progress-technologies.net> to control@bugs.debian.org. (Mon, 06 May 2013 12:45:08 GMT) Full text and rfc822 format available.

Marked Bug as done Request was from Daniel Baumann <daniel.baumann@progress-technologies.net> to control@bugs.debian.org. (Mon, 06 May 2013 12:45:09 GMT) Full text and rfc822 format available.

Notification sent to Andrew Jeffery <andrew.jeffery@ultra-avalon.com>:
Bug acknowledged by developer. (Mon, 06 May 2013 12:45:10 GMT) Full text and rfc822 format available.

Send a report that this bug log contains spam.


Debian bug tracking system administrator <owner@bugs.debian.org>. Last modified: Sun Apr 20 19:27:40 2014; Machine Name: buxtehude.debian.org

Debian Bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.