Re: SVN r310931 Bad code

> On Dec 31, 2016, at 06:14, Michael Butler <imb_at_protected-networks.net> wrote:
> 
> At line 1949 of head/contrib/bsnmp/lib/snmpclient.c, you changed ..
> 
> -		if ((sc->chost = malloc(strlen(s) + 1)) == NULL) {
> +		if ((sc->chost = strdup(strlen(s))) == NULL) {
> 
> This can't work as intended since strlen returns the length of the
> string not a pointer to it.
> 
> I expect this should be ..
> 
> 		if ((sc->chost = strdup(s)) == NULL) {

	Yeah… egg on my face... I fixed it in r310942.
Sorry for the breakage :/…
-Ngie