gsmartcontrol doesn't report ATA/SATA version correctly

Bug #1310282 reported by Burt P.
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gsmartcontrol (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Ubuntu 14.04
GSmartControl 0.8.7 (package version 0.8.7-1)
smartmontools release 6.2 dated 2013-07-26 at 17:38:20 UTC (smartmontools package 6.2+svn3841-1.2)

GSmartControl doesn't parse the ATA and SATA version from the output of smartctl correctly.
The ATA or SATA version is always reported as 0.

The attached screenshot shows the smartctl output and the GSmartControl information window.

ProblemType: Bug
DistroRelease: Ubuntu 14.04
Package: gsmartcontrol 0.8.7-1
ProcVersionSignature: Ubuntu 3.13.0-24.46-generic 3.13.9
Uname: Linux 3.13.0-24-generic x86_64
NonfreeKernelModules: nvidia
ApportVersion: 2.14.1-0ubuntu3
Architecture: amd64
CurrentDesktop: Unity
Date: Sun Apr 20 08:55:20 2014
InstallationDate: Installed on 2014-01-04 (105 days ago)
InstallationMedia: Ubuntu 13.10 "Saucy Salamander" - Release amd64 (20131016.1)
SourcePackage: gsmartcontrol
UpgradeStatus: Upgraded to trusty on 2014-03-09 (42 days ago)

Related branches

Revision history for this message
Burt P. (pburt0) wrote :
Revision history for this message
Burt P. (pburt0) wrote :

Version is '0' because the program tries to make an integer out of the value. Matching /ATA Version/ also matched "SATA Version" and caused the two instances of "ATA Version: 0" to appear. The attached change fixes those problems.

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "patch to smartctl_parser.cpp" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
Burt P. (pburt0) wrote :

The author tells me that "all these issues were resolved a while ago in the SVN version."
http://sourceforge.net/p/gsmartcontrol/code/HEAD/tree/trunk/gsmartcontrol/src/applib/smartctl_parser.cpp

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

This bug was fixed in the package gsmartcontrol - 0.8.7-1ubuntu1

---------------
gsmartcontrol (0.8.7-1ubuntu1) utopic; urgency=medium

  * Add 04_compat_with_smartmontools_6.patch. Fixes a crash at
    startup. LP: #1231229
  * Add 05_fix_ATA_version.patch. Correctly report the ATA/SATA version
    LP: #1310282
 -- Jackson Doak <email address hidden> Mon, 09 Jun 2014 19:37:24 +1000

Changed in gsmartcontrol (Ubuntu):
status: New → 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.