summaryrefslogtreecommitdiffstats
path: root/contrib/gcclibs/libiberty/objalloc.c
diff options
context:
space:
mode:
authorpfg <pfg@FreeBSD.org>2016-06-03 21:37:24 +0000
committerpfg <pfg@FreeBSD.org>2016-06-03 21:37:24 +0000
commit356a290b3ddb28e5a3a577d0bbc5307a6d39d3ef (patch)
treee0fc9db09cbcd591373c5dc40d1527ad2cd2ebc8 /contrib/gcclibs/libiberty/objalloc.c
parent7e8741e191563206554511078725b2502d5aa3a9 (diff)
downloadFreeBSD-src-356a290b3ddb28e5a3a577d0bbc5307a6d39d3ef.zip
FreeBSD-src-356a290b3ddb28e5a3a577d0bbc5307a6d39d3ef.tar.gz
libiberty: prevent integer overflow.
Take care of very old bug leading to heap-buffer overflow by processing certain file headers via bfd binary. PR: 200888 Obtained from: OpenBSD MFC after: 2 weeks
Diffstat (limited to 'contrib/gcclibs/libiberty/objalloc.c')
-rw-r--r--contrib/gcclibs/libiberty/objalloc.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/contrib/gcclibs/libiberty/objalloc.c b/contrib/gcclibs/libiberty/objalloc.c
index 3ddac2c..394b3af 100644
--- a/contrib/gcclibs/libiberty/objalloc.c
+++ b/contrib/gcclibs/libiberty/objalloc.c
@@ -1,5 +1,5 @@
/* objalloc.c -- routines to allocate memory for objects
- Copyright 1997 Free Software Foundation, Inc.
+ Copyright 1997-2012 Free Software Foundation, Inc.
Written by Ian Lance Taylor, Cygnus Solutions.
This program is free software; you can redistribute it and/or modify it
@@ -112,8 +112,10 @@ objalloc_create (void)
/* Allocate space from an objalloc structure. */
PTR
-_objalloc_alloc (struct objalloc *o, unsigned long len)
+_objalloc_alloc (struct objalloc *o, unsigned long original_len)
{
+ unsigned long len = original_len;
+
/* We avoid confusion from zero sized objects by always allocating
at least 1 byte. */
if (len == 0)
@@ -121,6 +123,11 @@ _objalloc_alloc (struct objalloc *o, unsigned long len)
len = (len + OBJALLOC_ALIGN - 1) &~ (OBJALLOC_ALIGN - 1);
+ /* CVE-2012-3509: Check for overflow in the alignment operation above
+ * and then malloc argument below. */
+ if (len + CHUNK_HEADER_SIZE < original_len)
+ return NULL;
+
if (len <= o->current_space)
{
o->current_ptr += len;
OpenPOWER on IntegriCloud