> On Dec 31, 2016, at 06:14, Michael Butler <i...@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

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to