Thanks for noticing this. I believe this one is safe, also. The code isn't super clear, but from what I can tell x->data will never be null. So in this case the check for null is not needed. I removed it from the im.pidgin.pidgin branch (aka trunk). Thanks!