diff options
| author | Rasmus Lerdorf <rasmus@lerdorf.com> | 2013-10-19 19:25:47 -0700 |
|---|---|---|
| committer | Rasmus Lerdorf <rasmus@lerdorf.com> | 2013-10-19 19:25:47 -0700 |
| commit | 6650bed41cf26818c73e09fcb1a0ae187ce1eeaa (patch) | |
| tree | 188add9ed1ffc13f0105232275c10d8220c839b0 | |
| parent | 259c74654cf05e136179ddd21a7a39d109cee2f3 (diff) | |
| parent | 1ce00048fde3dda1d75a6e47921e80c9a864d992 (diff) | |
| download | php-git-6650bed41cf26818c73e09fcb1a0ae187ce1eeaa.tar.gz | |
Merge branch 'PHP-5.5'
* PHP-5.5:
Fix unitialized opened_path here - found by Coverity
| -rw-r--r-- | ext/dba/dba.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/ext/dba/dba.c b/ext/dba/dba.c index 50a94dd2ad..ced90f0c62 100644 --- a/ext/dba/dba.c +++ b/ext/dba/dba.c @@ -625,7 +625,8 @@ static void php_dba_open(INTERNAL_FUNCTION_PARAMETERS, int persistent) char *file_mode; char mode[4], *pmode, *lock_file_mode = NULL; int persistent_flag = persistent ? STREAM_OPEN_PERSISTENT : 0; - char *opened_path, *lock_name; + char *opened_path = NULL; + char *lock_name; if(ac < 2) { WRONG_PARAM_COUNT; @@ -848,8 +849,10 @@ static void php_dba_open(INTERNAL_FUNCTION_PARAMETERS, int persistent) if (!persistent) { info->lock.name = opened_path; } else { - info->lock.name = pestrdup(opened_path, persistent); - efree(opened_path); + if (opened_path) { + info->lock.name = pestrdup(opened_path, persistent); + efree(opened_path); + } } } } |