> > > diff --git a/src/acm.c b/src/acm.c > > > index 8f147ef..fae9d0b 100644 > > > --- a/src/acm.c > > > +++ b/src/acm.c > > > @@ -193,7 +193,7 @@ static int log_level = 0; static char > > > lock_file[128] = "/var/run/ibacm.pid"; static short server_port = > > > 6125; static int support_ips_in_addr_cfg = 0; -static char > > > *prov_lib_path = IBACM_LIB_PATH; > > > +static char prov_lib_path[256] = IBACM_LIB_PATH; > > > > 256 is too short for a path. Typical is 4096 but would be nice if it > were > > dynamic. > > Oh and do you need the length as a #define somewhere to prevent overflowing > the buffer when reading from the config file?
The code only reads in values from the config file that are up to 256 bytes long. Have you seen paths longer than 256 in practice? -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
