You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by ji...@apache.org on 2007/10/05 14:30:53 UTC

svn commit: r582228 - /apr/apr-util/trunk/buckets/apr_buckets_alloc.c

Author: jim
Date: Fri Oct  5 05:30:53 2007
New Revision: 582228

URL: http://svn.apache.org/viewvc?rev=582228&view=rev
Log:
We should really check these

Modified:
    apr/apr-util/trunk/buckets/apr_buckets_alloc.c

Modified: apr/apr-util/trunk/buckets/apr_buckets_alloc.c
URL: http://svn.apache.org/viewvc/apr/apr-util/trunk/buckets/apr_buckets_alloc.c?rev=582228&r1=582227&r2=582228&view=diff
==============================================================================
--- apr/apr-util/trunk/buckets/apr_buckets_alloc.c (original)
+++ apr/apr-util/trunk/buckets/apr_buckets_alloc.c Fri Oct  5 05:30:53 2007
@@ -85,6 +85,9 @@
     apr_memnode_t *block;
 
     block = apr_allocator_alloc(allocator, ALLOC_AMT);
+    if (!block) {
+        return NULL;
+    }
     list = (apr_bucket_alloc_t *)block->first_avail;
     list->pool = NULL;
     list->allocator = allocator;
@@ -127,6 +130,9 @@
             endp = active->first_avail + SMALL_NODE_SIZE;
             if (endp >= active->endp) {
                 list->blocks = apr_allocator_alloc(list->allocator, ALLOC_AMT);
+                if (!list->blocks) {
+                    return NULL;
+                }
                 list->blocks->next = active;
                 active = list->blocks;
                 endp = active->first_avail + SMALL_NODE_SIZE;
@@ -140,6 +146,9 @@
     }
     else {
         apr_memnode_t *memnode = apr_allocator_alloc(list->allocator, size);
+        if (!memnode) {
+            return NULL;
+        }
         node = (node_header_t *)memnode->first_avail;
         node->alloc = list;
         node->memnode = memnode;