Remove an unsafe programming practice from test-fdleak; add XX comments It's a bad idea to have a read callback read a fixed amount; Instead, we should drain it, or have some way to know that more data is arrived. Not unsafe in this case, but let's not encourage people to learn bad habits. This commit also notes some other stuff to fix.