[wp-trac] [WordPress Trac] #50228: Plugins can inadvertently trigger `wp_die()` infinite loop during site creation

[WordPress Trac]

#50228: Plugins can inadvertently trigger `wp_die()` infinite loop during site
creation
--------------------------+-----------------------------
 Reporter:  iandunn       |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  General       |    Version:
 Severity:  normal        |   Keywords:
  Focuses:                |
--------------------------+-----------------------------
 ==== Problem

 This is one example of an execution path that leads to an infinite loop,
 but there are probably several others. It can happen to any callback that
 tries to query a new site's tables after the `wp_insert_site` action, and
 before the `make_db_current_silent()` call inside `wp_initialize_site()`.

 1. Creating a site in the UI or programatically calls eventually
 `wp_initialize_site()`
 1. That calls `switch_to_blog()`
 1. A plugin could hook into the `switch_blog()` action, and call
 `get_locale()`.
 1. That calls `get_option()`, which will trigger a MySQL error, because
 the tables don't exist.
 1. `wpdb->print_error( )` calls `_default_wp_die_handler()`
 1. That calls `get_language_attributes()`, which calls `get_option()`
 1. The process repeats until PHP gives up.

 > ( ! ) Fatal error: Uncaught Error: Maximum function nesting level of
 '256' reached, aborting! in wp-includes/class-wp-hook.php on line 196

 It's not obvious to plugin developers that this kind of thing can happen,
 so I think Core should handle it gracefully.

 Related: #49263

 ==== Potential solutions

 1. `wp_die()` coule call a new `_installing_wp_die_handler()` when
 `is_installing()` is `true`. This could be a minimal version of
 `_default_wp_die_handler()`, where no database queries are
 made.[[br]][[br]]
 1. `_default_wp_die_handler()` could avoid calling
 `get_language_attributes()` when  `is_installing()` is `true`.[[br]][[br]]
 1. Something else?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/50228>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform