Bump a bit the coverage

* `setcookie` doesn't always return `true` anymore
* clang-format
* Cookies with invalid decryption are dropped, but the request isn't anymore
* faulty unserialize are now dumpable

1 files changed, 5 insertions, 4 deletions

diff --git a/src/sp_session.c b/src/sp_session.c
index 0f1c0b8..73fda22 100644
--- a/src/sp_session.c
+++ b/src/sp_session.c
@@ -27,10 +27,11 @@ static ZEND_INI_MH((*old_OnUpdateSaveHandler)) = NULL;
 
 static int sp_hook_s_read(PS_READ_ARGS) {
   int r = old_s_read(mod_data, key, val, maxlifetime);
-  const sp_config_session* config_session = SNUFFLEUPAGUS_G(config).config_session;
+  const sp_config_session *config_session =
+      SNUFFLEUPAGUS_G(config).config_session;
 
-  if (r == SUCCESS && config_session->encrypt &&
-      val != NULL && *val != NULL && ZSTR_LEN(*val)) {
+  if (r == SUCCESS && config_session->encrypt && val != NULL && *val != NULL &&
+      ZSTR_LEN(*val)) {
     zend_string *orig_val = *val;
     zval val_zval;
     ZVAL_PSTRINGL(&val_zval, ZSTR_VAL(*val), ZSTR_LEN(*val));
@@ -40,7 +41,7 @@ static int sp_hook_s_read(PS_READ_ARGS) {
       if (config_session->simulation) {
         return ret;
       } else {
-        sp_terminate();
+        zend_bailout();
       }
     }