iohandlers: fix issue with qemu_set_fd_handler()
As spotted by Aneesh, some users pass a NULL opaque so we need to be more aggressive in checking whether a user means to unregister. Also fix a double free caused by tag not being reset to zero after delete. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
		
							parent
							
								
									344eecf699
								
							
						
					
					
						commit
						c82dc29a91
					
				| 
						 | 
				
			
			@ -93,10 +93,6 @@ static gboolean fd_trampoline(GIOChannel *chan, GIOCondition cond, gpointer opaq
 | 
			
		|||
{
 | 
			
		||||
    IOTrampoline *tramp = opaque;
 | 
			
		||||
 | 
			
		||||
    if (tramp->opaque == NULL) {
 | 
			
		||||
        return FALSE;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if ((cond & G_IO_IN) && tramp->fd_read) {
 | 
			
		||||
        tramp->fd_read(tramp->opaque);
 | 
			
		||||
    }
 | 
			
		||||
| 
						 | 
				
			
			@ -119,9 +115,10 @@ int qemu_set_fd_handler(int fd,
 | 
			
		|||
    if (tramp->tag != 0) {
 | 
			
		||||
        g_io_channel_unref(tramp->chan);
 | 
			
		||||
        g_source_remove(tramp->tag);
 | 
			
		||||
        tramp->tag = 0;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (opaque) {
 | 
			
		||||
    if (fd_read || fd_write || opaque) {
 | 
			
		||||
        GIOCondition cond = 0;
 | 
			
		||||
 | 
			
		||||
        tramp->fd_read = fd_read;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue