qemu-option: Don't reinvent append_option_parameters()
parse_option_parameters() may need to create a new option parameter list from a template list. Use append_option_parameters() instead of duplicating the code. Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
		
							parent
							
								
									eec77d9e71
								
							
						
					
					
						commit
						898c257ba8
					
				| 
						 | 
				
			
			@ -416,20 +416,13 @@ QEMUOptionParameter *parse_option_parameters(const char *param,
 | 
			
		|||
    char value[256];
 | 
			
		||||
    char *param_delim, *value_delim;
 | 
			
		||||
    char next_delim;
 | 
			
		||||
    size_t num_options;
 | 
			
		||||
 | 
			
		||||
    if (list == NULL) {
 | 
			
		||||
        return NULL;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (dest == NULL) {
 | 
			
		||||
        // Count valid options
 | 
			
		||||
        num_options = count_option_parameters(list);
 | 
			
		||||
 | 
			
		||||
        // Create a copy of the option list to fill in values
 | 
			
		||||
        dest = qemu_mallocz((num_options + 1) * sizeof(QEMUOptionParameter));
 | 
			
		||||
        allocated = dest;
 | 
			
		||||
        memcpy(dest, list, (num_options + 1) * sizeof(QEMUOptionParameter));
 | 
			
		||||
        dest = allocated = append_option_parameters(NULL, list);
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    while (*param) {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue