Hi, > Add 'break' for this case when dest is NULL (no > dests). Then we have to add {} for all branches as per > CodingStyle: > > if (!is_unavailable(dest)) > return dest; > if (dest) { > IP_VS_DBG_BUF > } else { > IP_VS_DBG > break; > } So if dest is NULL, there is no point trying to choose a different server? Alex -- To unsubscribe from this list: send the line "unsubscribe lvs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html