| commit | 63662139e519ce06090b2759cf4a1d291b9cc0e2 | [log] [tgz] |
|---|---|---|
| author | David Woodhouse <David.Woodhouse@intel.com> | Thu Mar 14 13:23:11 2013 +0000 |
| committer | David Woodhouse <David.Woodhouse@intel.com> | Mon Mar 18 11:40:21 2013 +0000 |
| tree | 65fe475f0f13719da4f78a85d44116a5c23ba7d1 | |
| parent | fe9ab00f8354a4c388e30301859c5741590c3809 [diff] |
params: Fix potential memory leak in add_sysfs_param() On allocation failure, it would fail to free the old attrs array which was no longer referenced by anything (since it would free the old module_param_attrs struct on the way out). Comment the suspicious-looking krealloc() usage to explain why it *isn't* actually buggy, despite looking like a classic realloc() usage bug. Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>