mirror of
				https://github.com/GrapheneOS/hardened_malloc.git
				synced 2025-11-03 01:06:33 +01:00 
			
		
		
		
	test: add tests for zero sized realloc(3)
C23 declared calling realloc(3) with a non-NULL pointer and zero size Undefined behavior. Check that hardened_malloc handles that case sanely by free'ing the old pointer and returning a special pointer, like `malloc(3)` called with size zero.
This commit is contained in:
		
							parent
							
								
									1d7fc7ffe0
								
							
						
					
					
						commit
						9ca3279507
					
				
					 6 changed files with 80 additions and 1 deletions
				
			
		
							
								
								
									
										3
									
								
								test/.gitignore
									
										
									
									
										vendored
									
									
								
							
							
						
						
									
										3
									
								
								test/.gitignore
									
										
									
									
										vendored
									
									
								
							| 
						 | 
					@ -41,4 +41,7 @@ overflow_small_8_byte
 | 
				
			||||||
uninitialized_read_large
 | 
					uninitialized_read_large
 | 
				
			||||||
uninitialized_read_small
 | 
					uninitialized_read_small
 | 
				
			||||||
realloc_init
 | 
					realloc_init
 | 
				
			||||||
 | 
					realloc_c23_undefined_behaviour
 | 
				
			||||||
 | 
					realloc_c23_undefined_behaviour_double_free
 | 
				
			||||||
 | 
					realloc_c23_undefined_behaviour_use_after_free
 | 
				
			||||||
__pycache__/
 | 
					__pycache__/
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -67,7 +67,10 @@ EXECUTABLES := \
 | 
				
			||||||
    invalid_malloc_object_size_small \
 | 
					    invalid_malloc_object_size_small \
 | 
				
			||||||
    invalid_malloc_object_size_small_quarantine \
 | 
					    invalid_malloc_object_size_small_quarantine \
 | 
				
			||||||
    impossibly_large_malloc \
 | 
					    impossibly_large_malloc \
 | 
				
			||||||
    realloc_init
 | 
					    realloc_init \
 | 
				
			||||||
 | 
					    realloc_c23_undefined_behaviour \
 | 
				
			||||||
 | 
					    realloc_c23_undefined_behaviour_double_free \
 | 
				
			||||||
 | 
					    realloc_c23_undefined_behaviour_use_after_free
 | 
				
			||||||
 | 
					
 | 
				
			||||||
all: $(EXECUTABLES)
 | 
					all: $(EXECUTABLES)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
							
								
								
									
										19
									
								
								test/realloc_c23_undefined_behaviour.c
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										19
									
								
								test/realloc_c23_undefined_behaviour.c
									
										
									
									
									
										Normal file
									
								
							| 
						 | 
					@ -0,0 +1,19 @@
 | 
				
			||||||
 | 
					#include <stdio.h>
 | 
				
			||||||
 | 
					#include <stdlib.h>
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					#include "test_util.h"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					OPTNONE int main(void) {
 | 
				
			||||||
 | 
					    char *p, *q, *r;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    p = malloc(16);
 | 
				
			||||||
 | 
					    if (!p) {
 | 
				
			||||||
 | 
					        return 1;
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    q = realloc(p, 0);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    free(q);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    return 0;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
							
								
								
									
										19
									
								
								test/realloc_c23_undefined_behaviour_double_free.c
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										19
									
								
								test/realloc_c23_undefined_behaviour_double_free.c
									
										
									
									
									
										Normal file
									
								
							| 
						 | 
					@ -0,0 +1,19 @@
 | 
				
			||||||
 | 
					#include <stdio.h>
 | 
				
			||||||
 | 
					#include <stdlib.h>
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					#include "test_util.h"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					OPTNONE int main(void) {
 | 
				
			||||||
 | 
					    char *p, *q, *r;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    p = malloc(16);
 | 
				
			||||||
 | 
					    if (!p) {
 | 
				
			||||||
 | 
					        return 1;
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    q = realloc(p, 0);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    free(p);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    return 0;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
							
								
								
									
										21
									
								
								test/realloc_c23_undefined_behaviour_use_after_free.c
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										21
									
								
								test/realloc_c23_undefined_behaviour_use_after_free.c
									
										
									
									
									
										Normal file
									
								
							| 
						 | 
					@ -0,0 +1,21 @@
 | 
				
			||||||
 | 
					#include <stdio.h>
 | 
				
			||||||
 | 
					#include <stdlib.h>
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					#include "test_util.h"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					OPTNONE int main(void) {
 | 
				
			||||||
 | 
					    char *p, *q, *r;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    p = malloc(256 * 1024);
 | 
				
			||||||
 | 
					    if (!p) {
 | 
				
			||||||
 | 
					        return 1;
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    q = realloc(p, 0);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    printf("%c\n", *p);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    free(q);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    return 0;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
| 
						 | 
					@ -169,6 +169,20 @@ class TestSimpleMemoryCorruption(unittest.TestCase):
 | 
				
			||||||
        self.assertEqual(stderr.decode("utf-8"),
 | 
					        self.assertEqual(stderr.decode("utf-8"),
 | 
				
			||||||
                         "fatal allocator error: invalid realloc\n")
 | 
					                         "fatal allocator error: invalid realloc\n")
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    def test_realloc_c23_undefined_behaviour(self):
 | 
				
			||||||
 | 
					        _stdout, stderr, returncode = self.run_test("realloc_c23_undefined_behaviour")
 | 
				
			||||||
 | 
					        self.assertEqual(returncode, 0)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    def test_realloc_c23_undefined_behaviour_double_free(self):
 | 
				
			||||||
 | 
					        _stdout, stderr, returncode = self.run_test("realloc_c23_undefined_behaviour_double_free")
 | 
				
			||||||
 | 
					        self.assertEqual(returncode, -6)
 | 
				
			||||||
 | 
					        self.assertEqual(stderr.decode("utf-8"),
 | 
				
			||||||
 | 
					                         "fatal allocator error: double free (quarantine)\n")
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    def test_realloc_c23_undefined_behaviour_use_after_free(self):
 | 
				
			||||||
 | 
					        _stdout, stderr, returncode = self.run_test("realloc_c23_undefined_behaviour_use_after_free")
 | 
				
			||||||
 | 
					        self.assertEqual(returncode, -11)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def test_write_after_free_large_reuse(self):
 | 
					    def test_write_after_free_large_reuse(self):
 | 
				
			||||||
        _stdout, _stderr, returncode = self.run_test(
 | 
					        _stdout, _stderr, returncode = self.run_test(
 | 
				
			||||||
            "write_after_free_large_reuse")
 | 
					            "write_after_free_large_reuse")
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		
		Reference in a new issue