The test in the module for editing a comment improperly uses && and also it could check arg(2) which has to be numeric.

Finally, the comment may get deleted in between so it wouldn't load. In that case the system should return FALSE immediately (we could also do that with the node.)

See attached.

Support from Acquia helps fund testing for Drupal Acquia logo