Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.When using php 5.3.x, the following warning appears on top of drupal pages:
warning: Parameter 2 to securepages_link_alter() expected to be a reference, value given in /home/alfredo/Public/alfredo.pironti.eu/research/includes/common.inc on line 2839.
This is potentially a security issue too, because it shows the full-path of the drupal installation; this is why I am marking the bug as critical.
Please note that drupal 6 is php5.3 compatible, so it is expected that many users are encountering this problem.
Any workaround while this issue gets solved? (So that the warning doesn't show up to users anymore)
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | securepages.module.patch | 443 bytes | alfredo.pironti |
Comments
Comment #1
alfredo.pironti commentedIndeed, the following patch fixes the issue.
Simply remove the & from the second parameter. Btw, the $node argument is never used, so this should not affect functionality in any way.
Comment #2
gordon commentedComment #4
SnakePick-1 commentedHi, I am getting this problem also with two warnings as follows: "warning: Parameter 2 to securepages_link_alter() expected to be a reference, value given in /home/snake185/public_html/includes/common.inc on line 2784." except I can't find any of the text on this line. I can't find the "&$node" anywhere in that file. Can you help me?