Found by the Coverity report, this one looks like a real bug:

check_provider_list has a if() branch to handle the passed-in 
current_provider being NULL, but never sets it to anything else; 
current_provider is later dereferenced unconditionally.

joe

Reply via email to