Acked-by: Jon Maloy <[email protected]> Thank you, Wei. > -----Original Message----- > From: Wei Yongjun [mailto:[email protected]] > Sent: Monday, March 26, 2018 10:33 > To: Jon Maloy <[email protected]>; Ying Xue > <[email protected]> > Cc: Wei Yongjun <[email protected]>; [email protected]; > [email protected]; [email protected] > Subject: [PATCH net-next] tipc: fix error handling in tipc_udp_enable() > > Release alloced resource before return from the error handling case in > tipc_udp_enable(), otherwise will cause memory leak. > > Fixes: 52dfae5c85a4 ("tipc: obtain node identity from interface by default") > Signed-off-by: Wei Yongjun <[email protected]> > --- > net/tipc/udp_media.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/net/tipc/udp_media.c b/net/tipc/udp_media.c index > 2c13b18..e7d91f5 100644 > --- a/net/tipc/udp_media.c > +++ b/net/tipc/udp_media.c > @@ -687,7 +687,8 @@ static int tipc_udp_enable(struct net *net, struct > tipc_bearer *b, > } > if (!tipc_own_id(net)) { > pr_warn("Failed to set node id, please configure > manually\n"); > - return -EINVAL; > + err = -EINVAL; > + goto err; > } > > b->bcast_addr.media_id = TIPC_MEDIA_TYPE_UDP;
------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ tipc-discussion mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/tipc-discussion
