Fix hang when completing a file transfer over XMPP

 






 

 


  Review Request #1466
  —
  Created May 21, 2022 and submitted  — Latest diff uploaded 

 


















 

  

   Information

  





 
 

  belgin
 









 
 

  pidgin/pidgin
 









 
 

  release-2.x.y
 









 
 

  
 









 
 

  
 






 




 

  

   Reviewers

  





 
 

  pidgin
 









 
 

  
 






 








 





  


 
 

  
The way purple_xfer_drain_socket was implemented,
if the peer never close()d the connection, we would
wait forever for it to be closed remotely, causing
Pidgin to hang.

 








  


 
 

  
Tested IRC and XMPP on Windows and Linux. Transferred big and small files.

 







 









 





  








 




 







   

    
    
Diff Revision 2


This is not the most recent revision of the diff. The latest diff is revision 4. See what's changed.

    

 

 

 

 
orig
1
2
3
4

 



    


    

      
Commits

      

 

 

  
   First
   Last
  
  Summary
  ID
  Author
 



 
  
   
  
  
   
  
 
 
  
   
    

     Fix hang when completing a file transfer over XMPP
     
The way purple_xfer_drain_socket was implemented,
if the peer never close()d the connection, we would
wait forever for it to be closed remotely, causing
Pidgin to hang.

    

   
  		
  58121aa13583f527b4f14b8f1dc89f7b2a9be83f
  Belgin Știrbu
 




    


    
    

   

  

 









 

  

   
    

     
      
      libpurple/ft.c
     
    

   
  

 



 

  

   
    

     
      
      libpurple/protocols/irc/dcc_send.c
     
    

   
  

 










     

    

   

  

  

   
   Loading...