[llvm-commits] [llvm] r108629 - /llvm/trunk/tools/llvmc/example/mcc16/plugins/PIC16Base/PluginMain.cpp

Duncan Sands baldrick at free.fr
Sat Jul 17 13:23:38 PDT 2010


Author: baldrick
Date: Sat Jul 17 15:23:37 2010
New Revision: 108629

URL: http://llvm.org/viewvc/llvm-project?rev=108629&view=rev
Log:
Fix what seems like a clear buffer overflow, noticed by cppcheck.

Modified:
    llvm/trunk/tools/llvmc/example/mcc16/plugins/PIC16Base/PluginMain.cpp

Modified: llvm/trunk/tools/llvmc/example/mcc16/plugins/PIC16Base/PluginMain.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvmc/example/mcc16/plugins/PIC16Base/PluginMain.cpp?rev=108629&r1=108628&r2=108629&view=diff
==============================================================================
--- llvm/trunk/tools/llvmc/example/mcc16/plugins/PIC16Base/PluginMain.cpp (original)
+++ llvm/trunk/tools/llvmc/example/mcc16/plugins/PIC16Base/PluginMain.cpp Sat Jul 17 15:23:37 2010
@@ -35,7 +35,7 @@
   }
 
   std::string LowerCase;
-  for (unsigned i = 0; i <= Partname.size(); i++) {
+  for (unsigned i = 0; i < Partname.size(); i++) {
     LowerCase.push_back(std::tolower(Partname[i]));
   }
 
@@ -52,7 +52,7 @@
   }
 
   std::string UpperCase;
-  for (unsigned i = 0; i <= Partname.size(); i++) {
+  for (unsigned i = 0; i < Partname.size(); i++) {
     UpperCase.push_back(std::toupper(Partname[i]));
   }
 





More information about the llvm-commits mailing list