[krbdev.mit.edu #3276] local array of structures not declared static

Denis Vlasenko via RT rt-comment at krbdev.mit.edu
Tue Dec 6 18:08:36 EST 2005


krb5-1.4.3/src/lib/kadm5/logger.c:

                       /*                         * If there is a facility present, then parse that.                         */                        if (cp2) {                            const struct {                                const char *name;                                int value;                            } facilities[] = {                                { "AUTH",       LOG_AUTH        },#ifdef  LOG_AUTHPRIV                                { "AUTHPRIV",   LOG_AUTHPRIV    },#endif  /* LOG_AUTHPRIV */#ifdef  LOG_KERN                                { "KERN",       LOG_KERN        },#endif  /* LOG_KERN */

Shouldn't we add "static" before "const" here?--vda



More information about the krb5-bugs mailing list