On Thu, 2012-03-29 at 12:03 +0300, Tanu Kaskinen wrote: > Coverity warned that r was possibly being used without > initialization. The fix is to add pa_assert_not_reached() > to handle the case where the address family is invalid. I > also felt that using switch would be more appropriate here > than using ifs and else-ifs, which makes this patch a bit > larger. I also changed the memset() calls to pa_zero(). > --- Looks good. -- Arun