Bug 12029 - libiodbc possible security issue (CVE-2013-7172)
Summary: libiodbc possible security issue (CVE-2013-7172)
Status: RESOLVED OLD
Alias: None
Product: Mageia
Classification: Unclassified
Component: Security (show other bugs)
Version: 3
Hardware: i586 Linux
Priority: Normal minor
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact: Sec team
URL: http://lwn.net/Vulnerabilities/577349/
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-18 00:37 CET by David Walser
Modified: 2014-11-27 15:55 CET (History)
2 users (show)

See Also:
Source RPM: libiodbc-3.52.8-6.mga4.src.rpm
CVE:
Status comment:


Attachments

Description David Walser 2013-12-18 00:37:10 CET
Slackware issued an advisory on December 16:
http://www.slackware.com/security/viewer.php?l=slackware-security&y=2013&m=slackware-security.480127

You can find the patch they applied (after ./configure) here:
http://mirrors.slackware.com/slackware/slackware-current/source/l/libiodbc/

It's an ugly way to do it, and you'd think it'd be better to patch configure, but it isn't clear which of the 10 instances of hardcode_libdir_flag_spec being defined with rpath in it you'd change to replicate their change to the "libtool" script.  It's also not clear to me what this vulnerability really is or if fixing this is necessary.

Reproducible: 

Steps to Reproduce:
David Walser 2013-12-18 00:37:28 CET

Whiteboard: (none) => MGA3TOO

Comment 1 David Walser 2013-12-19 14:00:57 CET
More info on this:
http://openwall.com/lists/oss-security/2013/12/19/2
Comment 2 David Walser 2013-12-20 13:54:42 CET
A CVE was assigned for this:
http://openwall.com/lists/oss-security/2013/12/20/1

Summary: libiodbc possible security issue => libiodbc possible security issue (CVE-2013-7172)

David Walser 2013-12-20 23:25:48 CET

Blocks: (none) => 11726

Comment 3 Oden Eriksson 2013-12-23 14:11:56 CET
I'd say this is invalid, but I fixed it anyway.

$ objdump -x /usr/bin/iodbctest |grep RPATH
  RPATH                /home/iurt/rpmbuild/BUILD/libiodbc-3.52.7/iodbc/.libs
$ objdump -x /usr/bin/iodbctestw |grep RPATH
  RPATH                /home/iurt/rpmbuild/BUILD/libiodbc-3.52.7/iodbc/.libs

Fixed with libiodbc-3.52.8-2.1.mga3 + libiodbc-3.52.8-7.mga4

CC: (none) => oe

Comment 4 David Walser 2013-12-23 14:20:34 CET
So is the issue that someone could put libraries in that RPATH directory and then libiodbc would load them?  I guess it'd be less of an issue for us since we don't use /tmp as the build directory as slackware apparently does, but I guess if you had a user named iurt it could still be an issue :o)
David Walser 2013-12-23 14:21:08 CET

Version: Cauldron => 3
Whiteboard: MGA3TOO => (none)

Comment 5 David Walser 2013-12-23 14:24:26 CET
(In reply to David Walser from comment #4)
> So is the issue that someone could put libraries in that RPATH directory and
> then libiodbc would load them?  I guess it'd be less of an issue for us
> since we don't use /tmp as the build directory as slackware apparently does,
> but I guess if you had a user named iurt it could still be an issue :o)

Not "libiodbc" would load them, but those two binaries I meant of course...
Comment 6 Oden Eriksson 2013-12-23 14:24:31 CET
Yes. Highly unlikely. I won't fox this for mbs1.
Comment 7 David Walser 2013-12-23 14:28:54 CET
Thanks Oden!

I think it'll be OK to leave this fixed in SVN and leave the bug open unless we have to issue an update for this package for some other reason later.

Severity: normal => minor

David Walser 2013-12-27 14:49:50 CET

Blocks: 11726 => (none)

Comment 8 Samuel Verschelde 2014-01-28 09:32:24 CET
Does comment #7 mean we can remove the update from updates_testing?

CC: (none) => stormi

Comment 9 David Walser 2014-01-28 12:35:32 CET
(In reply to Samuel VERSCHELDE from comment #8)
> Does comment #7 mean we can remove the update from updates_testing?

Can I suppose, though I don't see a need to.
Comment 10 Samuel Verschelde 2014-01-28 13:31:42 CET
(In reply to David Walser from comment #9)
> (In reply to Samuel VERSCHELDE from comment #8)
> > Does comment #7 mean we can remove the update from updates_testing?
> 
> Can I suppose, though I don't see a need to.

Taken alone, there's not a huge need to remove it, but cleaning the updates_testing repos from time to time helps to spot actual missed updates candidates. The ones where the package says "there's xxx.rpm to validate in updates_testing" and forgets (or doesn't know) the process involving QA.

The smaller the list of packages at http://mageia.madb.org/rpm/list/source/1/application/0/listtype/updates_testing the better :)
Comment 11 David Walser 2014-11-27 15:55:14 CET
Closing due to Mageia 3 EOL:
http://blog.mageia.org/en/2014/11/26/lets-say-goodbye-to-mageia-3/

Status: NEW => RESOLVED
Resolution: (none) => OLD


Note You need to log in before you can comment on or make changes to this bug.