Merge pull request #198 from samm-git/php7

check if session was allocated on PS_OPEN_FUNC

Author: mkoppanen

php_memcached_session.c

@@ -277,6 +277,11 @@ PS_OPEN_FUNC(memcached)
 PS_CLOSE_FUNC(memcached)
 {
 	memcached_sess *memc_sess = PS_GET_MOD_DATA();
+	
+	if (!memc_sess) {
+		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Session is not allocated, check session.save_path value");
+		return FAILURE;
+	}
 
 	if (MEMC_G(sess_locking_enabled)) {
 		php_memc_sess_unlock(memc_sess->memc_sess);
@@ -307,6 +312,11 @@ PS_READ_FUNC(memcached)
 	memcached_sess *memc_sess = PS_GET_MOD_DATA();
 	size_t key_length;
 
+	if (!memc_sess) {
+		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Session is not allocated, check session.save_path value");
+		return FAILURE;
+	}
+
 	key_length = strlen(MEMC_G(sess_prefix)) + key_len + 5; // prefix + "lock."
 	if (!key_length || key_length >= MEMCACHED_MAX_KEY) {
 		php_error_docref(NULL, E_WARNING, "The session id is too long or contains illegal characters");
@@ -339,6 +349,11 @@ PS_WRITE_FUNC(memcached)
 	memcached_return status;
 	memcached_sess *memc_sess = PS_GET_MOD_DATA();
 	size_t key_length;
+	
+	if (!memc_sess) {
+		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Session is not allocated, check session.save_path value");
+		return FAILURE;
+	}
 
 	key_length = strlen(MEMC_G(sess_prefix)) + key_len + 5; // prefix + "lock."
 	if (!key_length || key_length >= MEMCACHED_MAX_KEY) {
@@ -371,6 +386,11 @@ PS_DESTROY_FUNC(memcached)
 {
 	memcached_sess *memc_sess = PS_GET_MOD_DATA();
 
+	if (!memc_sess) {
+		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Session is not allocated, check session.save_path value");
+		return FAILURE;
+	}
+
 	memcached_delete(memc_sess->memc_sess, key->val, key->len, 0);
 	if (MEMC_G(sess_locking_enabled)) {
 		php_memc_sess_unlock(memc_sess->memc_sess);