Fix #80239: imap_rfc822_write_address() leaks memory

cmb69 · cmb69 · commit 5941b30bb889 · 2020-10-15T12:54:53.000+02:00
We have to free the address when we're finished with it.
diff --git a/NEWS b/NEWS
@@ -4,6 +4,7 @@ PHP                                                                        NEWS
 
 - IMAP:
   . Fixed bug #64076 (imap_sort() does not return FALSE on failure). (cmb)
+  . Fixed bug #80239 (imap_rfc822_write_address() leaks memory). (cmb)
 
 29 Oct 2020, PHP 7.3.24
 
diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c
@@ -2305,6 +2305,7 @@ PHP_FUNCTION(imap_subscribe)
 	} else {
 		RETURN_FALSE;
 	}
+	mail_free_address(&addr);
 }
 /* }}} */
 
diff --git a/ext/imap/tests/imap_rfc822_write_address_basic.phpt b/ext/imap/tests/imap_rfc822_write_address_basic.phpt
@@ -0,0 +1,12 @@
+--TEST--
+imap_rfc822_write_address() : basic functionality
+--SKIPIF--
+<?php
+if (!extension_loaded('imap')) die('skip imap extension not available');
+?>
+--FILE--
+<?php
+var_dump(imap_rfc822_write_address('me', 'example.com', 'My Name'));
+?>
+--EXPECT--
+string(24) "My Name <me@example.com>"
