diff options
| author | Matt Turner <mattst88@gmail.com> | 2010-06-04 21:09:35 -0400 |
|---|---|---|
| committer | Mikhail Gusarov <dottedmag@dottedmag.net> | 2010-06-11 19:05:46 +0700 |
| commit | 89bd05106e5823fc5cfca9abf082729f2444363b (patch) | |
| tree | 75b8ea2d3cb2e5640c02361d0e6317992939654b | |
| parent | a54a766dfb39fb3df671045878ac706215d83cef (diff) | |
record: move free() to after last use of pContext
No functional change, since free doesn't change the value of the pointer
passed to it, but it makes this code less confusing.
Reviewed-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Matt Turner <mattst88@gmail.com>
| -rw-r--r-- | record/record.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/record/record.c b/record/record.c index 53bdfec71..4117a5a02 100644 --- a/record/record.c +++ b/record/record.c @@ -2520,8 +2520,6 @@ RecordDeleteContext(pointer value, XID id) } } - free(pContext); - /* remove context from AllContexts list */ if (-1 != (i = RecordFindContextOnAllContexts(pContext))) @@ -2533,6 +2531,8 @@ RecordDeleteContext(pointer value, XID id) ppAllContexts = NULL; } } + free(pContext); + return Success; } /* RecordDeleteContext */ |