Fixed bug #68809 (Unchecked return value)

1 files changed, 8 insertions, 1 deletions

diff --git a/ext/phar/tar.c b/ext/phar/tar.c
index 6d3daf5c62..0195a515bd 100644
--- a/ext/phar/tar.c
+++ b/ext/phar/tar.c
@@ -481,7 +481,14 @@ bail:
 			entry.link = estrdup(hdr->linkname);
 		}
 		phar_set_inode(&entry);
-		newentry = zend_hash_str_add_mem(&myphar->manifest, entry.filename, entry.filename_len, (void*)&entry, sizeof(phar_entry_info));
+		if ((newentry = zend_hash_str_add_mem(&myphar->manifest, entry.filename, entry.filename_len, (void*)&entry, sizeof(phar_entry_info))) == NULL) {
+			if (error) {
+				spprintf(error, 4096, "phar error: tar-based phar \"%s\" cannot be registered", entry.filename);
+			}
+			php_stream_close(fp);
+			phar_destroy_phar_data(myphar);
+			return FAILURE;
+		}
 
 		if (entry.is_persistent) {
 			++entry.manifest_pos;