QMP: handle_qmp_command(): Small cleanup
Drop a unneeded label and QDECREF() call. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
This commit is contained in:
		
							parent
							
								
									0bbab46db6
								
							
						
					
					
						commit
						e4940c603a
					
				
							
								
								
									
										14
									
								
								monitor.c
									
									
									
									
									
								
							
							
						
						
									
										14
									
								
								monitor.c
									
									
									
									
									
								
							| @ -4234,7 +4234,7 @@ static void handle_qmp_command(JSONMessageParser *parser, QList *tokens) | ||||
|     Monitor *mon = cur_mon; | ||||
|     const char *cmd_name, *info_item; | ||||
| 
 | ||||
|     args = NULL; | ||||
|     args = input = NULL; | ||||
| 
 | ||||
|     obj = json_parser_parse(tokens, NULL); | ||||
|     if (!obj) { | ||||
| @ -4255,7 +4255,7 @@ static void handle_qmp_command(JSONMessageParser *parser, QList *tokens) | ||||
|     cmd_name = qdict_get_str(input, "execute"); | ||||
|     if (invalid_qmp_mode(mon, cmd_name)) { | ||||
|         qerror_report(QERR_COMMAND_NOT_FOUND, cmd_name); | ||||
|         goto err_input; | ||||
|         goto err_out; | ||||
|     } | ||||
| 
 | ||||
|     /*
 | ||||
| @ -4264,7 +4264,7 @@ static void handle_qmp_command(JSONMessageParser *parser, QList *tokens) | ||||
|      */ | ||||
|     if (compare_cmd(cmd_name, "info")) { | ||||
|         qerror_report(QERR_COMMAND_NOT_FOUND, cmd_name); | ||||
|         goto err_input; | ||||
|         goto err_out; | ||||
|     } else if (strstart(cmd_name, "query-", &info_item)) { | ||||
|         cmd = monitor_find_command("info"); | ||||
|         qdict_put_obj(input, "arguments", | ||||
| @ -4273,7 +4273,7 @@ static void handle_qmp_command(JSONMessageParser *parser, QList *tokens) | ||||
|         cmd = monitor_find_command(cmd_name); | ||||
|         if (!cmd || !monitor_handler_ported(cmd)) { | ||||
|             qerror_report(QERR_COMMAND_NOT_FOUND, cmd_name); | ||||
|             goto err_input; | ||||
|             goto err_out; | ||||
|         } | ||||
|     } | ||||
| 
 | ||||
| @ -4285,8 +4285,6 @@ static void handle_qmp_command(JSONMessageParser *parser, QList *tokens) | ||||
|         QINCREF(args); | ||||
|     } | ||||
| 
 | ||||
|     QDECREF(input); | ||||
| 
 | ||||
|     err = qmp_check_client_args(cmd, args); | ||||
|     if (err < 0) { | ||||
|         goto err_out; | ||||
| @ -4301,13 +4299,13 @@ static void handle_qmp_command(JSONMessageParser *parser, QList *tokens) | ||||
|     } else { | ||||
|         monitor_call_handler(mon, cmd, args); | ||||
|     } | ||||
| 
 | ||||
|     goto out; | ||||
| 
 | ||||
| err_input: | ||||
|     QDECREF(input); | ||||
| err_out: | ||||
|     monitor_protocol_emitter(mon, NULL); | ||||
| out: | ||||
|     QDECREF(input); | ||||
|     QDECREF(args); | ||||
| } | ||||
| 
 | ||||
|  | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user
	 Luiz Capitulino
						Luiz Capitulino