kernel-packages team mailing list archive

[Brad Figg <brad.figg@xxxxxxxxxxxxx>]

This bug is awaiting verification that the kernel in -proposed solves
the problem. Please test the kernel and update this bug with the
results. If the problem is solved, change the tag 'verification-needed-
utopic' to 'verification-done-utopic'.

If verification is not done by 5 working days from today, this fix will
be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how
to enable and use -proposed. Thank you!


** Tags added: verification-needed-utopic

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1434528

Title:
  [utopic] btrfs: label should not contain return char

Status in linux package in Ubuntu:
  Invalid
Status in linux source package in Utopic:
  Fix Committed

Bug description:
  SRU Justification:

  [Impact]
  Minor impact: It is possible to introduce a newline into the btrfs label output from the btrfs sys file /sys/fs/btrfs/$UUID/label, this newline should be filtered out.

  [Fix]
  clean upstream cherry pick 48fcc3ff7dce0138c053833adf81670494f177f3 "btrfs: label should not contain return char".

  [Testcase]
  The commit contains a suitable testcase, see below:

  TEST_DEV=/dev/vdb
  TEST_DIR=/home/sat/mnt

  umount /home/sat/mnt

  mkfs.btrfs -f $TEST_DEV
  UUID=$(btrfs fi show $TEST_DEV | head -1 | sed -e 's/.*uuid: \([-0-9a-z]*\)$/\1/')
  mount $TEST_DEV $TEST_DIR
  LABELFILE=/sys/fs/btrfs/$UUID/label

  echo "Test for empty label..." >&2
  LINES="$(cat $LABELFILE | wc -l | awk '{print $1}')"
  RET=0

  if [ $LINES -eq 0 ] ; then
      echo '[PASS] Trailing \n is removed correctly.' >&2
  else
      echo '[FAIL] Trailing \n still exists.' >&2
      RET=1
  fi

  echo "Test for non-empty label..." >&2

  echo testlabel >$LABELFILE
  LINES="$(cat $LABELFILE | wc -l | awk '{print $1}')"

  if [ $LINES -eq 1 ] ; then
      echo '[PASS] Trailing \n is removed correctly.' >&2
  else
      echo '[FAIL] Trailing \n still exists.' >&2
      RET=1
  fi

  exit $RET

  This is to be SRU'd for utopic only, trusty does not have this
  interface supported and this fix is carried in vivid.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1434528/+subscriptions