Actually, we prefer to have the loop-scope-only variables declared inside the loop rather than outside.1. I think "apr_bucket *bucket" might be better out of the while() loop rather than within the while(). But, with/without that change, it doesn't work for me. Rather, using inbio->bucket works.
I haven't tested the patch, so I can't speak to whether the whole thing is busted or not. => -- justin
