[SSSD] [PATCH] Add a default section to a swtich-statement

Simo Sorce simo at redhat.com
Wed Jan 9 13:34:32 UTC 2013


On Wed, 2013-01-09 at 10:51 +0100, Sumit Bose wrote:
> Hi,
> 
> a switch statement which was introduced with "Remove unhelpful vtable
> from sss_cache" has a missing default section which results in a
> compiler warning about an uninitialized variable. The attached patch
> should fix it.

The default section wasn't missing, but was explicitly left out. So that
if you add a new value to the enum then you start getting errors.

Please initialize ret before the switch instead.

(NACK)

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the sssd-devel mailing list