fix for using wxDataObjectComposite with the clipboard

[wxWidgets.git] / src / common / imagxpm.cpp

diff --git a/src/common/imagxpm.cpp b/src/common/imagxpm.cpp
index 7abef62368450f4e51da59ee044703984a636170..07f8ba9750f57d92f2850928b1dae0026ca04239 100644 (file)
--- a/src/common/imagxpm.cpp
+++ b/src/common/imagxpm.cpp
@@ -77,7 +77,7 @@ license is as follows:
 #  include "wx/defs.h"
 #endif
 
 #  include "wx/defs.h"
 #endif
 

-#if wxUSE_IMAGE && wxUSE_XPM
+#if wxUSE_XPM

 
 #include "wx/imagxpm.h"
 #include "wx/wfstream.h"
 
 #include "wx/imagxpm.h"
 #include "wx/wfstream.h"


@@ -94,9 +94,9 @@ IMPLEMENT_DYNAMIC_CLASS(wxXPMHandler,wxImageHandler)
 
 #if wxUSE_STREAMS
 
 
 #if wxUSE_STREAMS
 

-bool wxXPMHandler::LoadFile(wxImage *image, 
+bool wxXPMHandler::LoadFile(wxImage *image,

                             wxInputStream& stream,
                             wxInputStream& stream,

-                            bool verbose, int WXUNUSED(index))
+                            bool WXUNUSED(verbose), int WXUNUSED(index))

 {
     wxXPMDecoder decoder;
 
 {
     wxXPMDecoder decoder;
 


@@ -108,32 +108,36 @@ bool wxXPMHandler::LoadFile(wxImage *image,
 }
 
 bool wxXPMHandler::SaveFile(wxImage * image,
 }
 
 bool wxXPMHandler::SaveFile(wxImage * image,

-                            wxOutputStream& stream, bool verbose)
+                            wxOutputStream& stream, bool WXUNUSED(verbose))

 {
     wxString tmp;
     char tmp_c;
 {
     wxString tmp;
     char tmp_c;

-    
+

     // 1. count colours:
     #define MaxCixels  92
     // 1. count colours:
     #define MaxCixels  92

-    static const char Cixel[MaxCixels+1] = 
+    static const char Cixel[MaxCixels+1] =

                          " .XoO+@#$%&*=-;:>,<1234567890qwertyuipasdfghjk"
                          "lzxcvbnmMNBVCZASDFGHJKLPIUYTREWQ!~^/()_`'][{}|";
     int chars_per_pixel;
     int cols;
     int i, j, k;
                          " .XoO+@#$%&*=-;:>,<1234567890qwertyuipasdfghjk"
                          "lzxcvbnmMNBVCZASDFGHJKLPIUYTREWQ!~^/()_`'][{}|";
     int chars_per_pixel;
     int cols;
     int i, j, k;

-    
+

     cols = image->CountColours();
     chars_per_pixel = 1;
     for ( k = MaxCixels; cols > k; k *= MaxCixels)
         chars_per_pixel++;
 
     // 2. write the header:
     cols = image->CountColours();
     chars_per_pixel = 1;
     for ( k = MaxCixels; cols > k; k *= MaxCixels)
         chars_per_pixel++;
 
     // 2. write the header:

-    tmp.Printf("/* XPM */\n"
+    char tmpbuf[200];
+    // VS: 200b is safe upper bound for anything produced by sprintf bellow
+    //     (101 bytes the string, neither %i can expand into more than 10 chars)
+    sprintf(tmpbuf, 
+               "/* XPM */\n"

                "static char *xpm_data[] = {\n"
                "/* columns rows colors chars-per-pixel */\n"
                "static char *xpm_data[] = {\n"
                "/* columns rows colors chars-per-pixel */\n"

-               "\"%i %i %i %i\",\n", 
+               "\"%i %i %i %i\",\n",

                image->GetWidth(), image->GetHeight(), cols, chars_per_pixel);
                image->GetWidth(), image->GetHeight(), cols, chars_per_pixel);

-    stream.Write(tmp.mb_str(), tmp.Length());
+    stream.Write(tmpbuf, strlen(tmpbuf));

 
     // 3. create color symbols table:
     wxHashTable table(wxKEY_INTEGER);
 
     // 3. create color symbols table:
     wxHashTable table(wxKEY_INTEGER);


@@ -147,7 +151,7 @@ bool wxXPMHandler::SaveFile(wxImage * image,
     if (image->HasMask())
         mask_key = (image->GetMaskRed() << 16) |
                    (image->GetMaskGreen() << 8) | image->GetMaskBlue();
     if (image->HasMask())
         mask_key = (image->GetMaskRed() << 16) |
                    (image->GetMaskGreen() << 8) | image->GetMaskBlue();

-    
+

     // 2b. generate colour table:
     table.BeginFind();
     wxNode *node = NULL;
     // 2b. generate colour table:
     table.BeginFind();
     wxNode *node = NULL;


@@ -174,7 +178,7 @@ bool wxXPMHandler::SaveFile(wxImage * image,
         else if (key == mask_key)
             tmp.Printf(wxT("\"%s c None\",\n"), sym);
         else
         else if (key == mask_key)
             tmp.Printf(wxT("\"%s c None\",\n"), sym);
         else

-            tmp.Printf(wxT("\"%s c #%s%s%s\",\n"), sym, 
+            tmp.Printf(wxT("\"%s c #%s%s%s\",\n"), sym,

                        wxDecToHex((unsigned char)(key >> 16)).c_str(),
                        wxDecToHex((unsigned char)(key >> 8)).c_str(),
                        wxDecToHex((unsigned char)(key)).c_str());
                        wxDecToHex((unsigned char)(key >> 16)).c_str(),
                        wxDecToHex((unsigned char)(key >> 8)).c_str(),
                        wxDecToHex((unsigned char)(key)).c_str());


@@ -203,7 +207,7 @@ bool wxXPMHandler::SaveFile(wxImage * image,
     }
     tmp = wxT("};\n");
     stream.Write(tmp.mb_str(), 3);
     }
     tmp = wxT("};\n");
     stream.Write(tmp.mb_str(), 3);

-    
+

     delete[] symbols;
     delete[] symbols_data;
 
     delete[] symbols;
     delete[] symbols_data;
 


@@ -218,4 +222,4 @@ bool wxXPMHandler::DoCanRead(wxInputStream& stream)
 
 #endif  // wxUSE_STREAMS
 
 
 #endif  // wxUSE_STREAMS
 

-#endif // wxUSE_XPM && wxUSE_IMAGE
+#endif // wxUSE_XPM