]>
Commit | Line | Data |
---|---|---|
e5fd101c PS |
1 | autofs-5.0.4 - fix double free in expire_proc() |
2 | ||
3 | From: Ian Kent <raven@themaw.net> | |
4 | ||
5 | In state.c:expire_proc() the function expire_proc_cleanup() is called | |
6 | which frees the parameter structure but automount frees this again in | |
7 | the following line. | |
8 | --- | |
9 | ||
10 | CHANGELOG | 1 + | |
11 | daemon/state.c | 1 - | |
12 | 2 files changed, 1 insertions(+), 1 deletions(-) | |
13 | ||
14 | ||
15 | diff --git a/CHANGELOG b/CHANGELOG | |
16 | index 2cb35dc..8860b2c 100644 | |
17 | --- a/CHANGELOG | |
18 | +++ b/CHANGELOG | |
19 | @@ -28,6 +28,7 @@ | |
20 | - add nfs mount protocol default configuration option. | |
21 | - fix bad token declaration in master map parser. | |
22 | - fix return start status on fail. | |
23 | +- fix double free in expire_proc(). | |
24 | ||
25 | 4/11/2008 autofs-5.0.4 | |
26 | ----------------------- | |
27 | diff --git a/daemon/state.c b/daemon/state.c | |
28 | index 606743b..417fde1 100644 | |
29 | --- a/daemon/state.c | |
30 | +++ b/daemon/state.c | |
31 | @@ -298,7 +298,6 @@ static enum expire expire_proc(struct autofs_point *ap, int now) | |
32 | error(ap->logopt, | |
33 | "expire thread create for %s failed", ap->path); | |
34 | expire_proc_cleanup((void *) ea); | |
35 | - free(ea); | |
36 | return EXP_ERROR; | |
37 | } | |
38 | ap->exp_thread = thid; |