Skip to content

Commit

Permalink
In mmc_flush_handler the return param is a pointer to integer and ZVA…
Browse files Browse the repository at this point in the history
…L_FALSE should not be used as it results in memory corruption. The failure case is not supposed to modify the return param. (#97)
  • Loading branch information
vnsavage authored Jul 29, 2022
1 parent ccf702b commit 96910ce
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/memcache.c
Original file line number Diff line number Diff line change
Expand Up @@ -2449,7 +2449,6 @@ static int mmc_flush_handler(mmc_t *mmc, mmc_request_t *request, int response, c
}

if (response == MMC_RESPONSE_CLIENT_ERROR) {
ZVAL_FALSE((zval *)param);
php_error_docref(NULL, E_NOTICE,
"Server %s (tcp %d, udp %d) failed with: %s (%d)",
mmc->host, mmc->tcp.port,
Expand Down

0 comments on commit 96910ce

Please sign in to comment.