I've been coding modules using the current available API and I've recently tried compiling PHP-5.1.2 for Apache 2.0 and I'm not obtaining the expected results when a function that should return an array but doesn't.

Compiling PHP 5.1.2 (or any previous version) for Apache 1.3.x the module works properly and the issue only occurs when compiling for 2.0.

I'm not sure if there are API changes that are not in the current documentation so if someone could examine the following code and sees something obvious I'd appreciate it if you'd point it out.

-- Dale



PHP_FUNCTION(ram_disk_file_load)
{
        pval *file, *args, *p_argc;
        pval *entry;
        zval *p_prcalc;
        HashTable *args_arr;
        int i, xsize, ysize, argc;
        double ymin,ymax;
        char **argv, **prcalc;

        /*
         * xsize is the number of bytes per sector
         * ysize is the number of bytes (in kb) for the ramdisk.
         * argv[0] is the command to perform.
         * argv[1] is either 'disk' mode or 'raw' mode
* argv[2] contains the file with the disk images contents to load into it
         */

        if ( disk_test_error() )
                disk_clear_error();

if ( (ZEND_NUM_ARGS() >= 3 && ZEND_NUM_ARGS() <= 6) && zend_get_parameters(ht, 3, &file, &args, &p_argc) == SUCCESS)
        {
                if ( args->type != IS_ARRAY )
                {
php_error(E_WARNING, "2nd Variable passed to disk_config is not an array!\n");
                        RETURN_FALSE;
                }

                convert_to_long(p_argc);
                convert_to_string(file);

                convert_to_array(args);
                args_arr = args->value.ht;

                // here we want to raw copy our saved content file to the new 
ram disk
                argc = p_argc->value.lval + 3;
                argv = (char **) emalloc(argc * sizeof(char *));
                argv[0] = "init";
                argv[1] = estrdup("raw");
                argv[2] = estrdup(file->value.str.val);

                for (i = 3; i < argc; i++)
                {
                        pval **dataptr;

if ( zend_hash_get_current_data(args_arr, (void *) &dataptr) == FAILURE )
                                continue;

                        entry = *dataptr;

                        if ( entry->type != IS_STRING )
                                convert_to_string(entry);

                        argv[i] = estrdup(entry->value.str.val);

                        if ( i < argc )
                                zend_hash_move_forward(args_arr);
                }

                optind = 0; opterr = 0;
                if (size _config(argc-1, &argv[1], &prcalc, &xsize, &ysize) != 
-1 )
                {
                        array_init(return_value);
                        add_assoc_long(return_value, "xsize", xsize);
                        add_assoc_long(return_value, "ysize", ysize);

                        MAKE_STD_ZVAL(p_prcalc);
                        array_init(p_prcalc);

                        if (prcalc)
                        {
                                for (i = 0; prcalc[i]; i++)
                                {
                                        add_next_index_string(p_prcalc, 
prcalc[i], 1);
                                        free(prcalc[i]);
                                }
                                free(prcalc);
                        }
                        zend_hash_update(return_value->value.ht, "prcalc", 
sizeof("prcalc"),
                                                        (void *)&p_prcalc, 
sizeof(zval *), NULL);
                }
                else
                {
                        RETVAL_FALSE; // PROBLEM - FAILED TO PERFORM THE TASK
                }
                for (i = 1; i < argc; i++)
                        efree(argv[i]);

                efree(argv);
        }
        else
        {
                WRONG_PARAM_COUNT;
        }
        return;
}

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to