config: Constify the public interface.
The two methods, config_create_interface and config_read, never modify the strings passed in. This patch adds the const keyword to ensure these functions stay that way. Signed-off-by: Richard Cochran <richardcochran@gmail.com> Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>master
parent
b8480ef908
commit
831f7e073b
4
config.c
4
config.c
|
@ -710,7 +710,7 @@ static struct option *config_alloc_longopts(void)
|
||||||
return opts;
|
return opts;
|
||||||
}
|
}
|
||||||
|
|
||||||
int config_read(char *name, struct config *cfg)
|
int config_read(const char *name, struct config *cfg)
|
||||||
{
|
{
|
||||||
enum config_section current_section = UNKNOWN_SECTION;
|
enum config_section current_section = UNKNOWN_SECTION;
|
||||||
enum parser_result parser_res;
|
enum parser_result parser_res;
|
||||||
|
@ -817,7 +817,7 @@ parse_error:
|
||||||
return -2;
|
return -2;
|
||||||
}
|
}
|
||||||
|
|
||||||
struct interface *config_create_interface(char *name, struct config *cfg)
|
struct interface *config_create_interface(const char *name, struct config *cfg)
|
||||||
{
|
{
|
||||||
struct interface *iface;
|
struct interface *iface;
|
||||||
|
|
||||||
|
|
4
config.h
4
config.h
|
@ -60,8 +60,8 @@ struct config {
|
||||||
STAILQ_HEAD(ucmtab_head, unicast_master_table) unicast_master_tables;
|
STAILQ_HEAD(ucmtab_head, unicast_master_table) unicast_master_tables;
|
||||||
};
|
};
|
||||||
|
|
||||||
int config_read(char *name, struct config *cfg);
|
int config_read(const char *name, struct config *cfg);
|
||||||
struct interface *config_create_interface(char *name, struct config *cfg);
|
struct interface *config_create_interface(const char *name, struct config *cfg);
|
||||||
void config_destroy(struct config *cfg);
|
void config_destroy(struct config *cfg);
|
||||||
|
|
||||||
/* New, hash table based methods: */
|
/* New, hash table based methods: */
|
||||||
|
|
Loading…
Reference in New Issue