This was not introduced in this change.

On Mon, Sep 18, 2017 at 8:05 AM, Hans Petter Selasky <h...@selasky.org> wrote:
> On 09/18/17 01:40, Conrad Meyer wrote:
>>
>>         device_get_children(dev, &children, &nchildren);
>>         for (i = 0; i < nchildren; i++) {
>>                 if (children[i])
>> -                       linsysfs_run_bus(children[i], dir, scsi, new_path,
>> prefix);
>> +                       linsysfs_run_bus(children[i], dir, scsi, chardev,
>> new_path, prefix);
>>         }
>>         if (new_path != path)
>>                 free(new_path, M_TEMP);
>> +       free(chardevname, M_TEMP);
>>         return (1);
>
>
> 1) Return code from device_get_children() should be checked.
>
> 2) children pointer should be freed else there is a memory leak.
>
> --HPS
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to