diff options
| author | Jukka Rissanen <jukka.rissanen@linux.intel.com> | 2011-08-31 11:42:44 +0300 |
|---|---|---|
| committer | Samuel Ortiz <sameo@linux.intel.com> | 2011-09-12 11:40:48 +0200 |
| commit | cbb7cb383df22e3e027c1339c0211e808d941e13 (patch) | |
| tree | 866dc64f42b51e8715766f6ac9277354daf7fc54 /src/timezone.c | |
| parent | 4140b53b867a8ceb35be039a8baf5c9f6e2e3434 (diff) | |
| download | connman-cbb7cb383df22e3e027c1339c0211e808d941e13.tar.gz connman-cbb7cb383df22e3e027c1339c0211e808d941e13.tar.bz2 connman-cbb7cb383df22e3e027c1339c0211e808d941e13.zip | |
timezone: Do not access already released memory
closedir() releases the filename but we were accessing it later.
Diffstat (limited to 'src/timezone.c')
| -rw-r--r-- | src/timezone.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/timezone.c b/src/timezone.c index 08f904f5..1234622c 100644 --- a/src/timezone.c +++ b/src/timezone.c @@ -185,9 +185,10 @@ static char *find_origin(void *src_map, struct stat *src_st, subpath, d->d_name); if (compare_file(src_map, src_st, pathname) == 0) { - closedir(dir); - return g_strdup_printf("%s/%s", + str = g_strdup_printf("%s/%s", subpath, d->d_name); + closedir(dir); + return str; } break; case DT_DIR: |