Thread (1 message) 1 message, 1 author, 2019-03-19

Re: [PATCH] net-sysfs: Fix memory leak in netdev_register_kobject

From: Stephen Hemminger <stephen@networkplumber.org>
Date: 2019-03-19 15:45:02
Also in: lkml

On Tue, 19 Mar 2019 20:17:01 +0800
"wanghai (M)" [off-list ref] wrote:
+out:
         return error;
+register_error:
+       device_del(dev);
+device_add_error:
+       put_device(dev);
+       goto out;
Everything looks fine, but I would redo the last part without
the last goto out. Using a goto back to single return reduces
readability.

	if (error)
               goto register_error;
        
	pm_runtime_set_memalloc_noio(dev, true);
	return 0;

register_error:
      device_del(dev);
device_add_error:
       put_device(dev);
       return error;
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help