Syntax error extra parenthesis linux-headers-3.13.0-100/Makefile

Bug #1636625 reported by Gary
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Undecided
Unassigned
Trusty
Fix Released
Undecided
Tim Gardner

Bug Description

Recommended change: remove incorrect extra ) character

root@nereid:/usr/src/linux-headers-3.13.0-100# diff Makefile old2.Makefile
615c615
< -fstack-protector not supported by compiler)
---
> -fstack-protector not supported by compiler))

This bug causes driver builds to fail like this

make[1]: Entering directory `/usr/src/linux-headers-3.13.0-100-generic'
/usr/src/linux-headers-3.13.0-100-generic/arch/x86/Makefile:111: CONFIG_X86_X32 enabled but no binutils support
Makefile:614: Cannot use CONFIG_CC_STACKPROTECTOR: -fstack-protector not supported by compiler
Makefile:614: *** missing separator. Stop.
make[1]: Leaving directory `/usr/src/linux-headers-3.13.0-100-generic'
make: *** [all] Error 2
---
ApportVersion: 2.14.1-0ubuntu3.18
Architecture: amd64
AudioDevicesInUse: Error: command ['fuser', '-v', '/dev/snd/seq', '/dev/snd/timer'] failed with exit code 1:
CRDA: Error: [Errno 2] No such file or directory
DistroRelease: Ubuntu 14.04
HibernationDevice: RESUME=UUID=93497dab-63bb-4998-9e07-583b6fbd6493
InstallationDate: Installed on 2014-09-03 (784 days ago)
InstallationMedia: Ubuntu-Server 14.04 LTS "Trusty Tahr" - Release amd64 (20140416.2)
IwConfig:
 bond0 no wireless extensions.

 eth0 no wireless extensions.

 lo no wireless extensions.
MachineType: HP ProLiant ML110 G5
Package: linux (not installed)
ProcEnviron:
 TERM=xterm
 PATH=(custom, no user)
 XDG_RUNTIME_DIR=<set>
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcFB: 0 VESA VGA
ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.13.0-100-generic root=UUID=2083fcba-62cb-427d-abb2-cacc50e14374 ro text nomdmonddf nomdmonisw
ProcVersionSignature: Ubuntu 3.13.0-100.147-generic 3.13.11-ckt39
PulseList: Error: command ['pacmd', 'list'] failed with exit code 1: No PulseAudio daemon running, or not running as session daemon.
RelatedPackageVersions:
 linux-restricted-modules-3.13.0-100-generic N/A
 linux-backports-modules-3.13.0-100-generic N/A
 linux-firmware 1.127.22
RfKill:

Tags: trusty
Uname: Linux 3.13.0-100-generic x86_64
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups:

WifiSyslog:

_MarkForUpload: True
dmi.bios.date: 10/25/2010
dmi.bios.vendor: HP
dmi.bios.version: O15
dmi.board.asset.tag: No Asset Tag
dmi.board.name: ProLiant ML110 G5
dmi.board.vendor: Wistron Corporation
dmi.board.version: NA
dmi.chassis.asset.tag: No Asset Tag
dmi.chassis.type: 7
dmi.chassis.vendor: HP
dmi.chassis.version: N/A
dmi.modalias: dmi:bvnHP:bvrO15:bd10/25/2010:svnHP:pnProLiantML110G5:pvrNA:rvnWistronCorporation:rnProLiantML110G5:rvrNA:cvnHP:ct7:cvrN/A:
dmi.product.name: ProLiant ML110 G5
dmi.product.version: NA
dmi.sys.vendor: HP

CVE References

Revision history for this message
Gary (islandlife) wrote :

The warning statement parenthesis is not balanced

ifdef CONFIG_CC_STACKPROTECTOR_REGULAR
  stackp-flag := -fstack-protector
  ifeq ($(call cc-option, $(stackp-flag)),)
    $(warning Cannot use CONFIG_CC_STACKPROTECTOR: \
              -fstack-protector not supported by compiler))
  endif
else ifdef CONFIG_CC_STACKPROTECTOR_STRONG
  stackp-flag := -fstack-protector-strong
  ifeq ($(call cc-option, $(stackp-flag)),)
    $(warning Cannot use CONFIG_CC_STACKPROTECTOR_STRONG: \
              -fstack-protector-strong not supported by compiler)
  endif
else
  # Force off for distro compilers that enable stack protector by default.
  stackp-flag := $(call cc-option, -fno-stack-protector)
endif
KBUILD_CFLAGS += $(stackp-flag)

Revision history for this message
Brad Figg (brad-figg) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. From a terminal window please run:

apport-collect 1636625

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
tags: added: trusty
Revision history for this message
Gary (islandlife) wrote :

Kernel (and kernel headers) linux-headers-3.13.0-100-generic was released
in response to CVE 2016-5195

It is then unfortunate that customers upgrading urgently to kernel 3.13.0-100-generic in Ubuntu 14.04.1 LTS can not rebuild some customer-specific drivers because of the syntax error in this Makefile :-(

Revision history for this message
Gary (islandlife) wrote : AlsaInfo.txt

apport information

tags: added: apport-collected
description: updated
Revision history for this message
Gary (islandlife) wrote : BootDmesg.txt

apport information

Revision history for this message
Gary (islandlife) wrote : CurrentDmesg.txt

apport information

Revision history for this message
Gary (islandlife) wrote : Lspci.txt

apport information

Revision history for this message
Gary (islandlife) wrote : Lsusb.txt

apport information

Revision history for this message
Gary (islandlife) wrote : ProcCpuinfo.txt

apport information

Revision history for this message
Gary (islandlife) wrote : ProcInterrupts.txt

apport information

Revision history for this message
Gary (islandlife) wrote : ProcModules.txt

apport information

Revision history for this message
Gary (islandlife) wrote : UdevDb.txt

apport information

Revision history for this message
Gary (islandlife) wrote : UdevLog.txt

apport information

Revision history for this message
Gary (islandlife) wrote :

Is it not complete now?

Changed in linux (Ubuntu):
status: Incomplete → New
Revision history for this message
Brad Figg (brad-figg) wrote : Status changed to Confirmed

This change was made by a bot.

Changed in linux (Ubuntu):
status: New → Confirmed
Revision history for this message
Tim Gardner (timg-tpi) wrote :
Changed in linux (Ubuntu Trusty):
assignee: nobody → Tim Gardner (timg-tpi)
status: New → In Progress
Changed in linux (Ubuntu):
status: Confirmed → Fix Released
Tim Gardner (timg-tpi)
Changed in linux (Ubuntu Trusty):
status: In Progress → Fix Committed
Revision history for this message
Luis Henriques (henrix) wrote :

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-trusty' to 'verification-done-trusty'.

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-trusty
Tim Gardner (timg-tpi)
tags: added: verification-done-trusty
removed: verification-needed-trusty
Revision history for this message
Steve Langasek (vorlon) wrote : Update Released

The verification of the Stable Release Update for linux-lts-trusty has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package linux - 3.13.0-103.150

---------------
linux (3.13.0-103.150) trusty; urgency=low

  [ Luis Henriques ]

  * Release Tracking Bug
    - LP: #1644489

  * Possible regression on 3.13.0-102.149~precise1 x86_64 (gce) (LP: #1644302)
    - SAUCE: apparmor: delete extra variable dev_path

linux (3.13.0-102.149) trusty; urgency=low

  [ Luis Henriques ]

  * Release Tracking Bug
    - LP: #1640581

  * lxc-attach to malicious container allows access to host (LP: #1639345)
    - Revert "UBUNTU: ptrace: being capable wrt a process requires mapped
      uids/gids"
    - (upstream) mm: Add a user_ns owner to mm_struct and fix ptrace permission
      checks

  * Syntax error extra parenthesis linux-headers-3.13.0-100/Makefile
    (LP: #1636625)
    - Makefile: fix extra parenthesis typo when CC_STACKPROTECTOR_REGULAR is
      enabled

  * Add a driver for Amazon Elastic Network Adapters (ENA) (LP: #1635721)
    - lib/bitmap.c: conversion routines to/from u32 array
    - kernel.h: define u8, s8, u32, etc. limits
    - net: ethtool: add new ETHTOOL_xLINKSETTINGS API
    - PCI/MSI: Add pci_msix_vec_count()
    - etherdevice: Use ether_addr_copy to copy an Ethernet address
    - net: ena: Add a driver for Amazon Elastic Network Adapters (ENA)
    - [config] enable CONFIG_ENA_ETHERNET=m (Amazon ENA driver)

  * CVE-2016-8658
    - brcmfmac: avoid potential stack overflow in brcmf_cfg80211_start_ap()

  * CVE-2016-7425
    - scsi: arcmsr: Buffer overflow in arcmsr_iop_message_xfer()

  * srcname from mount rule corrupted under load (LP: #1634753)
    - SAUCE: apparmor: fix sleep in critical section

  * ghash-clmulni-intel module fails to load (LP: #1633058)
    - crypto: ghash-clmulni - Fix load failure
    - crypto: cryptd - Assign statesize properly

 -- Luis Henriques <email address hidden> Thu, 24 Nov 2016 09:56:54 +0000

Changed in linux (Ubuntu Trusty):
status: Fix Committed → 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.