]> granicus.if.org Git - php/commitdiff
- Fix valgrind reports + CS
authorFelipe Pena <felipensp@gmail.com>
Mon, 18 Nov 2013 21:55:29 +0000 (19:55 -0200)
committerFelipe Pena <felipensp@gmail.com>
Mon, 18 Nov 2013 21:55:29 +0000 (19:55 -0200)
phpdbg.c
phpdbg_list.c
phpdbg_print.c

index eaa626d39a776e31c995f8ebbce9133ac23a366f..6659075a0da61e5e672282b49a19db714b4becd3 100644 (file)
--- a/phpdbg.c
+++ b/phpdbg.c
@@ -41,6 +41,7 @@ static inline void php_phpdbg_globals_ctor(zend_phpdbg_globals *pg) /* {{{ */
     pg->lcmd = NULL;
     pg->flags = PHPDBG_DEFAULT_FLAGS;
     pg->oplog = NULL;
+    memset(&pg->lparam, 0, sizeof(phpdbg_param_t));
 } /* }}} */
 
 static PHP_MINIT_FUNCTION(phpdbg) /* {{{ */
@@ -94,14 +95,14 @@ static void php_phpdbg_destroy_bp_condition(void *data) /* {{{ */
 } /* }}} */
 
 static PHP_RINIT_FUNCTION(phpdbg) /* {{{ */
-{      
+{
        zend_hash_init(&PHPDBG_G(bp)[PHPDBG_BREAK_FILE],   8, NULL, php_phpdbg_destroy_bp_file, 0);
        zend_hash_init(&PHPDBG_G(bp)[PHPDBG_BREAK_SYM], 8, NULL, php_phpdbg_destroy_bp_symbol, 0);
     zend_hash_init(&PHPDBG_G(bp)[PHPDBG_BREAK_OPLINE], 8, NULL, NULL, 0);
     zend_hash_init(&PHPDBG_G(bp)[PHPDBG_BREAK_METHOD], 8, NULL, php_phpdbg_destroy_bp_methods, 0);
     zend_hash_init(&PHPDBG_G(bp)[PHPDBG_BREAK_COND], 8, NULL, php_phpdbg_destroy_bp_condition, 0);
        zend_hash_init(&PHPDBG_G(seek), 8, NULL, NULL, 0);
-       
+
        return SUCCESS;
 } /* }}} */
 
@@ -113,7 +114,7 @@ static PHP_RSHUTDOWN_FUNCTION(phpdbg) /* {{{ */
     zend_hash_destroy(&PHPDBG_G(bp)[PHPDBG_BREAK_METHOD]);
     zend_hash_destroy(&PHPDBG_G(bp)[PHPDBG_BREAK_COND]);
        zend_hash_destroy(&PHPDBG_G(seek));
-       
+
     if (PHPDBG_G(exec)) {
         efree(PHPDBG_G(exec));
         PHPDBG_G(exec) = NULL;
index dcce3746305ec5d4a4c9ec19b03f5b5c8849ee45..241983d742806dcce1c794e572bc8766b1abb8c9 100644 (file)
@@ -153,7 +153,7 @@ void phpdbg_list_file(const char *filename, long count, long offset, int highlig
                return;
        }
 
-       last_pos = mem = mmap(0, st.st_size, PROT_READ, MAP_SHARED, fd, 0);
+       pos = last_pos = mem = mmap(0, st.st_size, PROT_READ, MAP_SHARED, fd, 0);
        end_pos = mem + st.st_size;
 #else
 
@@ -170,7 +170,7 @@ void phpdbg_list_file(const char *filename, long count, long offset, int highlig
                return;
        }
 
-       last_pos = mem = (char*) MapViewOfFile(map, FILE_MAP_READ, 0, 0, 0);
+       pos = last_pos = mem = (char*) MapViewOfFile(map, FILE_MAP_READ, 0, 0, 0);
        if (mem == NULL) {
                phpdbg_error("Failed to map file in memory");
                CloseHandle(map);
index 606e091021a93592cb92a6bce4a4edb99ebe6abb..f3821bee1ea5c1e483d01edfa784f61c37760b23 100644 (file)
@@ -37,7 +37,8 @@ PHPDBG_PRINT(opline) /* {{{ */
        return SUCCESS;
 } /* }}} */
 
-static inline void phpdbg_print_function_helper(zend_function *method TSRMLS_DC) {
+static inline void phpdbg_print_function_helper(zend_function *method TSRMLS_DC) /* {{{ */
+{
     switch (method->type) {
         case ZEND_USER_FUNCTION: {
             zend_op_array* op_array = &method->op_array;
@@ -85,7 +86,7 @@ static inline void phpdbg_print_function_helper(zend_function *method TSRMLS_DC)
             }
         }
      }
-}
+} /* }}} */
 
 PHPDBG_PRINT(exec) /* {{{ */
 {