apache log showed a stack trace which had alwaysthis line

PHP 12. drupalforfirebug_array_compare_code() /home/xl/workspace/sanago/sites/all/modules/drupalforfirebug/drupalforfirebug.module:512, referer: http://default.sanago.localhost/

didn't get to further research yet

CommentFileSizeAuthor
#2 1129452.patch784 bytesgeek-merlin

Comments

geek-merlin’s picture

added a dirty debug line to that function:

/**
* Specialized Array Data Comparision Code
*/
function drupalforfirebug_array_compare_code($a, $b, $c = array(), $history = array()) {
  error_log(print_r(get_defined_vars(),1));

and squeezed out the input that brought the function to that infinite loop:

Array
(
    [a] => 
    [b] => Array
        (
            [js] => Array
                (
                    [0] => modules/menu/menu.js
                )

            [library] => Array
                (
                    [0] => Array
                        (
                            [0] => system
                            [1] => drupal.form
                        )

                    [1] => Array
                        (
                            [0] => system
                            [1] => drupal.collapse
                        )

                )

        )

    [c] => Array
        (
            [js] => Array
                (
                    [0] => modules/menu/menu.js
                )

            [library] => Array
                (
                    [0] => Array
                        (
                            [0] => system
                            [1] => drupal.form
                        )

                    [1] => Array
                        (
                            [0] => system
                            [1] => drupal.collapse
                        )

                )

        )

    [history] => Array
        (
        )

)


geek-merlin’s picture

Status: Active » Needs review
StatusFileSize
new784 bytes

tralala, found a commit that removed recursion conditions: 14b2bdb109e9337063d45a8cd805a842c23e045b

reverted that part an voila it goes!

populist’s picture

Status: Needs review » Fixed

Thanks for looking into this. The recursion issues are a little tricky, but hopefully this works for most people.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.