Skip to content

Commit

Permalink
Terminate smart string correctly
Browse files Browse the repository at this point in the history
This is related to Zend VM regression bug #75881.  The regression was fixed in the VM, so there is no user visible change from the termination correction.
  • Loading branch information
cjbj committed Sep 24, 2018
1 parent 91954c2 commit 9e56a33
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ext/oci8/oci8.c
Original file line number Diff line number Diff line change
Expand Up @@ -1788,12 +1788,12 @@ php_oci_connection *php_oci_do_connect_ex(char *username, int username_len, char
timestamp = time(NULL);

smart_str_append_unsigned_ex(&hashed_details, session_mode, 0);
smart_str_0(&hashed_details);

if (persistent) {
smart_str_appendl_ex(&hashed_details, "pc", sizeof("pc") - 1, 0);
}

smart_str_0(&hashed_details);

/* make it lowercase */
php_strtolower(ZSTR_VAL(hashed_details.s), ZSTR_LEN(hashed_details.s));

Expand Down

0 comments on commit 9e56a33

Please sign in to comment.