Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140111205823.GA16165@eldamar.local>
Date: Sat, 11 Jan 2014 21:58:23 +0100
From: Salvatore Bonaccorso <carnil@...ian.org>
To: oss-security@...ts.openwall.com
Subject: Re: CVE assignment for jinja2

Hi Vicnent,

Disclaimer: to be taken with some caution.

On Sat, Jan 11, 2014 at 01:37:51PM -0700, Vincent Danen wrote:
> On 01/10/2014, at 22:34 PM, Kurt Seifried wrote:
> 
> > https://github.com/mitsuhiko/jinja2/commit/acb672b6a179567632e032f547582f30fa2f4aa7
> >
> > dirname = '_jinja2-cache-%d' % os.getuid()
> >
> > Arun Babu Neelicattu of Red Hat spotted this commit which introduces a
> > temporary file creation vulnerability. This issue has been assigned
> > CVE-2014-0012. For information on how to safely create temporary files
> > please see
> > http://kurt.seifried.org/2012/03/14/creating-temporary-files-securely/
> >
> > For Python simply use ?mkstemp? for files and ?mkdtemp? for
> > directories from the ?tempfile? module.
> 
> MITRE assigned CVE-2014-1402 to this yesterday:
> 
> http://seclists.org/oss-sec/2014/q1/71 (the report, the followup has the CVE assignment).
> 
> That means you'll need to reject this assignment; the commit that Arun spotted was due to the Debian bug report (which the git commit notes, and Ratul linked to in his initial CVE request to the list).

Aren't the two CVE assignments correct this way as the second
temporary file creation vulnerability was introduced by the mentioned
commit?

Initially there was assigned CVE-2014-1402 for:

http://seclists.org/oss-sec/2014/q1/71

wich is also http://bugs.debian.org/734747 and was attempted to be
fixed with commit
https://github.com/mitsuhiko/jinja2/commit/acb672b6a179567632e032f547582f30fa2f4aa7

But the above commit introduces a new temporary file creation
vulnerability, which then got CVE-2014-0012 assigned by Kurt.

Regards,
Salvatore

Powered by blists - more mailing lists

Please check out the Open Source Software Security Wiki, which is counterpart to this mailing list.

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.