[patch] fix for BadAtom in X_ChangeProperty when pasting to some client

[Emanuele Giaquinta]

On Tue, Apr 19, 2011 at 10:44:08PM +0000, Marcin Cieslak wrote:
> Hello,
> 
> When testing syncterm (http://syncterm.bbsdev.net/) I noticed
> that syncterm crashes and (after some fixes) causes crash
> in rxvt and the following warning in rxvt-unicode when
> pasting some text from rxvt-unicode to syncterm:
> 
> X Error occurred, trying to continue after report.
> urxvt: X Error of failed request:  BadAtom (invalid Atom parameter)
> urxvt: Major opcode of failed request:  18
> urxvt: (which is X_ChangeProperty)
> urxvt: Atom id in failed request:  0x0
> urxvt: Serial number of failed request:  657
> 
> The same thing works fine with xterm. This is because
> this client sends None property with ConvertRequest. 
> This is discouradged by ICCCM but nethertheless I think
> it should not cause any problems with the terminal
> emulator.
> 
> Here are some more sample programs that work the
> same way:
> 
> http://michael.toren.net/mirrors/doc/X-copy+paste.txt
> 
> Below is a patch to rxvt-unicode-9.10 (it applies
> also to the CVS trunk):

Thanks, applied.

Emanuele