This fix is for the problem Matthew Hagerty reported. (PHP appearing to execute code...

I didn't hear back from Mr. Hagerty, so I'll assume the fix worked for him.
(It worked for me)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@4269 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/plugins/squirrelspell/modules/check_me.mod b/plugins/squirrelspell/modules/check_me.mod
index 41cfd15c8855cb6b28304d6d95e101b8e5825b48..06ffe25b2e32eb48c2761d3044f4ac9dfc74da8e 100644 (file)
--- a/plugins/squirrelspell/modules/check_me.mod
+++ b/plugins/squirrelspell/modules/check_me.mod
@@ -191,12 +191,16 @@ for ($i=0; $i<sizeof($sqspell_output); $i++){
      */
     if (!$SQSPELL_EREG("\n$sqspell_word\n", $words)){
       $sqspell_symb=intval($tmparray[2])-1;
-      if (!$misses[$sqspell_word]) {
-       $misses[$sqspell_word] = '_NONE';
-       $missed_words[$errors] = $sqspell_word;
-       $errors++;
+      if (!isset($misses[$sqspell_word])) {
+           $misses[$sqspell_word] = '_NONE';
+           $missed_words[$errors] = $sqspell_word;
+           $errors++;
       }
-      if ($locations[$sqspell_word]) $locations[$sqspell_word] .= ', ';
+      if (isset($locations[$sqspell_word])) {
+           $locations[$sqspell_word] .= ', ';
+      } else {
+           $locations[$sqspell_word] = '';
+         }
       $locations[$sqspell_word] .= "$current_line:$sqspell_symb";
     }
   break;