bugfix: EE_OPEN_WARNING could be statistically wrong

don't rely on imprecise my_file_opened | my_stream_opened,
scan the array for open handles instead.

also, remove my_print_open_files() and embed it in my_end()
to have one array scan instead of two.

1 files changed, 21 insertions, 5 deletions

diff --git a/mysys/my_init.c b/mysys/my_init.c
index aebcb3c1b8a..972750da0ff 100644
--- a/mysys/my_init.c
+++ b/mysys/my_init.c
@@ -158,17 +158,33 @@ void my_end(int infoflag)
   }
 
   if ((infoflag & MY_CHECK_ERROR) || print_info)
+  {                                     /* Test if some file is left open */
+    char ebuff[512];
+    uint i, open_files, open_streams;
 
-  {					/* Test if some file is left open */
-    if (my_file_opened | my_stream_opened)
+    for (open_streams= open_files= i= 0 ; i < my_file_limit ; i++)
+    {
+      if (my_file_info[i].type == UNOPEN)
+        continue;
+      if (my_file_info[i].type == STREAM_BY_FOPEN ||
+          my_file_info[i].type == STREAM_BY_FDOPEN)
+        open_streams++;
+      else
+        open_files++;
+
+#ifdef EXTRA_DEBUG
+      fprintf(stderr, EE(EE_FILE_NOT_CLOSED), my_file_info[i].name, i);
+      fputc('\n', stderr);
+#endif
+    }
+    if (open_files || open_streams)
     {
-      char ebuff[512];
       my_snprintf(ebuff, sizeof(ebuff), EE(EE_OPEN_WARNING),
-                  my_file_opened, my_stream_opened);
+                  open_files, open_streams);
       my_message_stderr(EE_OPEN_WARNING, ebuff, ME_BELL);
       DBUG_PRINT("error", ("%s", ebuff));
-      my_print_open_files();
     }
+
 #ifdef CHECK_UNLIKELY
     end_my_likely(info_file);
 #endif