Clarification on advisories
Werner Koch
wk at gnupg.org
Mon Mar 23 13:20:24 CET 2015
On Mon, 23 Mar 2015 11:05, venture37 at gmail.com said:
> Are the applicable parts of the issues highlighted here:
> http://www.openwall.com/lists/oss-security/2015/02/13/14
> Backported to 2.0.27?
Yes, all four:
1. 39978487863066e59bb657f5fe4e8baab510da7e
commit 7e12ec4c7d6df29a7d7935399fccd2594ebb4a7e
Author: Werner Koch <wk at gnupg.org>
Date: Thu Feb 12 18:52:07 2015 +0100
gpg: Fix a NULL-deref due to empty ring trust packets.
* g10/parse-packet.c (parse_trust): Always allocate a packet.
--
Reported-by: Hanno Böck <hanno at hboeck.de>
Signed-off-by: Werner Koch <wk at gnupg.org>
(back ported from commit 39978487863066e59bb657f5fe4e8baab510da7e)
2. 0835d2f44ef62eab51fce6a927908f544e01cf8f
commit 8da836e76f1349f4587d1bb74864b11dde7b8a39
Author: Werner Koch <wk at gnupg.org>
Date: Thu Feb 12 18:54:17 2015 +0100
gpg: Fix a NULL-deref in export due to invalid packet lengths.
* g10/build-packet.c (write_fake_data): Take care of a NULL stored as
opaque MPI.
--
Reported-by: Hanno Böck <hanno at hboeck.de>
(back ported from commit 0835d2f44ef62eab51fce6a927908f544e01cf8f)
3. 0f71a721ccd7ab9e40b8b6b028b59632c0cc648
commit 824d88ac51b4d680f06e68f0879a7c1ec03cb2ba
Author: Werner Koch <wk at gnupg.org>
Date: Thu Feb 12 18:58:36 2015 +0100
gpg: Prevent an invalid memory read using a garbled keyring.
* g10/keyring.c (keyring_get_keyblock): Whitelist allowed packet
types.
--
The keyring DB code did not reject packets which don't belong into a
keyring. If for example the keyblock contains a literal data packet
it is expected that the processing code stops at the data packet and
reads from the input stream which is referenced from the data packets.
Obviously the keyring processing code does not and cannot do that.
However, when exporting this messes up the IOBUF and leads to an
invalid read of sizeof (int).
We now skip all packets which are not allowed in a keyring.
Reported-by: Hanno Böck <hanno at hboeck.de>
(back ported from commit f0f71a721ccd7ab9e40b8b6b028b59632c0cc648)
4. 2183683bd633818dd031b090b5530951de76f392
commit 3627123dc8fdc551caca1c7944713fbf01feccf6
Author: Werner Koch <wk at gnupg.org>
Date: Thu Feb 12 20:34:44 2015 +0100
Use inline functions to convert buffer data to scalars.
* include/host2net.h (buf16_to_ulong, buf16_to_uint): New.
(buf16_to_ushort, buf16_to_u16): New.
(buf32_to_size_t, buf32_to_ulong, buf32_to_uint, buf32_to_u32): New.
--
This fixes sign extension on shift problems. Hanno Böck found a case
with an invalid read due to this problem. To fix that almost all uses
of "<< 24" and "<< 8" are changed by this patch to use an inline
function from host2net.h.
(back ported from commit 2183683bd633818dd031b090b5530951de76f392)
and releases with 2.0.27
commit 8d47e6e5235b6ecb41baf52865c5837c1de962b5
Author: Werner Koch <wk at gnupg.org>
Date: Wed Feb 18 14:10:57 2015 +0100
Release 2.0.27
Salam-Shalom,
Werner
--
Die Gedanken sind frei. Ausnahmen regelt ein Bundesgesetz.
More information about the Gnupg-users
mailing list