From 991ae6e7d3bde939df0abc69654236506d3210a3 Mon Sep 17 00:00:00 2001 From: Antony Dovgal Date: Mon, 18 Aug 2008 13:09:41 +0000 Subject: [PATCH] MFH: add note on 64bit compatibility and mention check_parameters.php --- README.PARAMETER_PARSING_API | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/README.PARAMETER_PARSING_API b/README.PARAMETER_PARSING_API index 1551e8e67e..77aa5c12a4 100644 --- a/README.PARAMETER_PARSING_API +++ b/README.PARAMETER_PARSING_API @@ -65,6 +65,32 @@ Type specifiers to all specifiers except for 'b', 'l', and 'd'). If NULL is passed, the results pointer is set to NULL as well. + +Note on 64bit compatibility +--------------------------- +Please do not forget that int and long are two different things on 64bit +OSes (int is 4bit and long is 8bit), so make sure you pass longs to "l" +and ints to strings length (i.e. for "s" you need to pass char * and int), +not the other way round! +Remember: "l" is the only case when you need to pass long (and that's why +it's "l", not "i" btw). + +Both mistakes cause memory corruptions and segfaults on 64bit OSes: +1) + char *str; + long str_len; /* XXX THIS IS WRONG!! Use int instead. */ + zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &str, &str_len) + +2) + int num; /* XXX THIS IS WRONG!! Use long instead. */ + zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &num) + +If you're in doubt, use check_parameters.php script to the parameters +and their types (it can be found in ./scripts/dev/ directory of PHP sources): + +# php ./scripts/dev/check_parameters.php /path/to/your/sources/ + + Examples -------- /* Gets a long, a string and its length, and a zval */ -- 2.40.0