From: Michael D. <md...@go...> - 2005-10-05 18:09:42
|
Fix memory leak in send_request() and remove an unnecessary type cast of the return value from malloc. Signed-off-by: Michael Davidson <md...@go...> --- relay-apps-0.8/librelay.c 2005-08-05 10:03:30.000000000 -0700 +++ relay-apps/librelay.c 2005-10-05 11:01:18.000000000 -0700 @@ -106,7 +106,7 @@ struct nlmsghdr *req; int err; - req = (struct nlmsghdr *)malloc(NLMSG_SPACE(len)); + req = malloc(NLMSG_SPACE(len)); memset(req, 0, NLMSG_SPACE(len)); req->nlmsg_len = NLMSG_LENGTH(len); req->nlmsg_type = type; @@ -117,6 +117,9 @@ err = send(control_channel, req, req->nlmsg_len, 0); if (err < 0) fprintf(stderr, "netlink send error\n"); + + free(req); + return err; } |