Free clientData when cleaning up client

`rfbClientSetClientData()` allocates a new `rfbClientData`, but never gets cleaned up, which causes memory leaks.

The proposed change fixes this by freeing all elements in the linked list in `rfbClientCleanup()`.