Messages in this thread | | | Date | Thu, 1 Oct 2015 18:04:50 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH] staging: dgap: fix memory leak in dgap_parsefile() |
| |
On Thu, Oct 01, 2015 at 09:11:31AM +0530, Ronit Halder wrote: > The existing dgap_parsefile() rejects invalid config file. > But before we know that config file is invalid a lot of memory leak > can happen. Removing the chances of memory leak won't heart anyone. >
Why not just reject it earlier instead of allocating more data?
regards, dan carpenter
| |