This caused flooding of my watchdog table with "headers already sent" type errors because bots were trying to access forward url's for unpublished nodes.

Patch will follow for 1.x-dev (what the site I'm working on is running). Issue may apply to newer versions too

Basically it's:
- return drupal_access_denied();
+ return MENU_ACCESS_DENIED;

Similar fixes have been committed to core recently:
http://drupalcode.org/project/drupal.git/commit/257fac8

CommentFileSizeAuthor
#1 fix_page_callback_not_found-2257049-1.patch1.17 KBrv0
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rv0’s picture

rv0 CreditAttribution: rv0 commented
Status: Active » Needs review
FileSize
fix_page_callback_not_found-2257049-1.patch1.17 KB

the patch also corrects 3 whitespace errors, can't help it, my IDE does this automagically ;)

  • john.oltman committed 89542ab on 7.x-2.x authored by rv0 
    Issue #2257049 by rv0: Page callbacks should not call...

  • john.oltman committed 21ecf0f on authored by rv0 
    Issue #2257049 by rv0: Page callbacks should not call...
john.oltman’s picture

john.oltman CreditAttribution: john.oltman commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Needs review » Fixed

Thanks for the patch

Committed to 7.x-2.x and 7.x-3.x

Use the patch for 7.x-1.x

Status: Fixed » Closed (fixed)

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