Skip Menu |
 

From: Denis Vlasenko <vda@ilport.com.ua>
To: krb5-bugs@mit.edu
Date: Tue, 6 Dec 2005 17:19:57 +0200
Subject: local array of structures not declared static
Download (untitled) / with headers
text/plain 1.2KiB
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
From: raeburn@mit.edu
Subject: SVN Commit
Make local const array of structures static too.

Commit By: raeburn



Revision: 18165
Changed Files:
U trunk/src/lib/kadm5/logger.c