softmmu-semi: fix lock_user* functions not to deref NULL upon OOM
Return NULL upon malloc failure. Signed-off-by: Jim Meyering <meyering@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
		
							parent
							
								
									396bef4b38
								
							
						
					
					
						commit
						15d9e3bc6a
					
				| 
						 | 
				
			
			@ -40,7 +40,7 @@ static void *softmmu_lock_user(CPUArchState *env, uint32_t addr, uint32_t len,
 | 
			
		|||
    uint8_t *p;
 | 
			
		||||
    /* TODO: Make this something that isn't fixed size.  */
 | 
			
		||||
    p = malloc(len);
 | 
			
		||||
    if (copy)
 | 
			
		||||
    if (p && copy)
 | 
			
		||||
        cpu_memory_rw_debug(env, addr, p, len, 0);
 | 
			
		||||
    return p;
 | 
			
		||||
}
 | 
			
		||||
| 
						 | 
				
			
			@ -52,6 +52,9 @@ static char *softmmu_lock_user_string(CPUArchState *env, uint32_t addr)
 | 
			
		|||
    uint8_t c;
 | 
			
		||||
    /* TODO: Make this something that isn't fixed size.  */
 | 
			
		||||
    s = p = malloc(1024);
 | 
			
		||||
    if (!s) {
 | 
			
		||||
        return NULL;
 | 
			
		||||
    }
 | 
			
		||||
    do {
 | 
			
		||||
        cpu_memory_rw_debug(env, addr, &c, 1, 0);
 | 
			
		||||
        addr++;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue