Changeset 6136

Timestamp:

04/09/08 01:02:44 (8 months ago)

Author:

flynd

Message:

Reverted r6135 as requested by emostar.

Location:

trunk/licq/src

Files:

• icqd-chat.cpp (modified) (4 diffs)
• icqd-srv.cpp (modified) (3 diffs)
• icqd-tcp.cpp (modified) (24 diffs)
• icqd.cpp (modified) (8 diffs)
• onevent.cpp (modified) (1 diff)

trunk/licq/src/icqd-chat.cpp

@@ -1292 +1292 @@
          sizeFontName = u->chatQueue[1] | (u->chatQueue[2] << 8);
          if (u->chatQueue.size() < (unsigned long)(sizeFontName + 2 + 3)) return true;
-         char* nameFont = new char[sizeFontName + 1];
+         char nameFont[sizeFontName + 1];
          for (i = 0; i < sizeFontName; i++)
             nameFont[i] = u->chatQueue[i + 3];
@@ -1306 +1306 @@
 
          PushChatEvent(new CChatEvent(CHAT_FONTxFAMILY, u));
-         delete [] nameFont;
          break;
       }
@@ -1606 +1605 @@
            unsigned short sizeFontName, i;
            sizeFontName = u->chatQueue[0] | (u->chatQueue[1] << 8);
-           char* nameFont = new char[sizeFontName + 1];
+           char nameFont[sizeFontName + 1];
            for (i = 0; i < sizeFontName; i++)
               nameFont[i] = u->chatQueue[i + 2];
@@ -1618 +1617 @@
            //the size includes the following character, so don't dequeue it
            chatSize--;
-           delete [] nameFont;
            break;
         }

trunk/licq/src/icqd-srv.cpp

@@ -3035 +3035 @@
       nMsgLen -= 4;
 
-        char* szMessage = new char[nMsgLen+1];
+      char* szMessage = new char[nMsgLen+1];
       for (int i = 0; i < nMsgLen; i++)
         szMessage[i] = msgTxt.UnpackChar();
@@ -3842 +3842 @@
     packet >> nAckFlags >> nMsgFlags >> nLen;
     
-    char* szMessage = new char[nLen + 1];
+    char szMessage[nLen + 1];
     for (unsigned short i = 0; i < nLen; i++)
       packet >> szMessage[i];
@@ -3886 +3886 @@
     }
     gUserManager.DropUser(u);
-
-    delete [] szMessage;
-
+    
     ICQEvent *e = DoneServerEvent(nMsgID, EVENT_ACKED);
     if (e)

trunk/licq/src/icqd-tcp.cpp

@@ -343 +343 @@
   gTranslator.ClientToServer(szDescDos);
   int n = strlen_safe(url) + strlen_safe(szDescDos) + 2;
-  char* m = new char[n];
+  char m[n];
   if (!online && n > MAX_MESSAGE_SIZE && szDescDos != NULL)
     szDescDos[MAX_MESSAGE_SIZE - strlen_safe(url) - 2] = '\0';
@@ -392 +392 @@
   if (szDescDos)
     delete [] szDescDos;
-
-  delete [] m;
 
   if (result != NULL)
@@ -1931 +1929 @@
   else
   {
-
+  
   // read in the message minus any stupid DOS \r's
-  char* messageTmp = new char[messageLen + 1];
+  char messageTmp[messageLen + 1];
   unsigned short j = 0;
   for (unsigned short i = 0; i < messageLen; i++)
@@ -1943 +1941 @@
 
   message = parseRTF(messageTmp);
-  delete [] messageTmp;
 
   if (nInVersion <= 4)
@@ -2321 +2318 @@
         packet >> junkLong
                >> nLenFilename;
-        char* szFilename = new char[nLenFilename+1];
+        char szFilename[nLenFilename+1];
         for (unsigned short i = 0; i < nLenFilename; i++)
            packet >> szFilename[i];
@@ -2354 +2351 @@
                                        filelist, theSequence, TIME_NOW,
                                        nMask | licqVersion);
-        delete [] szFilename;
         // Add the user to our list if they are new
         if (bNewUser)
@@ -2383 +2379 @@
                 packet >> nLongLen; // plugin name len
 
-                                char* szPlugin = new char[nLongLen+1];
+                char szPlugin[nLongLen+1];
                 for (unsigned long i = 0; i < nLongLen; i++)
                     packet >> szPlugin[i];
@@ -2404 +2400 @@
                 {
                     gLog.Info(tr("%sUnknown ICBM plugin type: %s\n"), L_TCPxSTR, szPlugin);
-          delete [] szPlugin;
-      break;
-    }
-
-        char* szMessage = new char[nLongLen+1];
+                    break;
+                }
+
+                char szMessage[nLongLen+1];
                 for (unsigned long i = 0; i < nLongLen; i++)
                     packet >> szMessage[i];
@@ -2421 +2416 @@
                     packet.incDataPosRead(2); // unknown
                     packet >> nLen; // filename len, including NULL
-            char* szFilename = new char[nLen+1];
+                    char szFilename[nLen+1];
                     for (unsigned short i = 0; i < nLen; i++)
                         packet >> szFilename[i];
@@ -2437 +2432 @@
                     gTranslator.ServerToClient(szMessage);
                     CEventFile *e = new CEventFile(szFilename, szMessage, nFileSize,
-                filelist, theSequence, TIME_NOW, nMask);
-            delete [] szFilename;
-
+                                                                                 filelist, theSequence, TIME_NOW, nMask);
                     if (bNewUser)
                     {
@@ -2551 +2544 @@
                 } // switch nICBMCommand
 
-        delete [] szPlugin;
-        delete [] szMessage;
-
-    break;
-      }
+                break;
+            }
 
       // Old-style encryption request:
@@ -2780 +2770 @@
                 packet >> nLongLen; // Plugin name len
 
-        char* szPlugin = new char[nLongLen+1];
+                char szPlugin[nLongLen+1];
                 for (unsigned long i = 0; i < nLongLen; i++)
                     packet >> szPlugin[i];
@@ -2802 +2792 @@
                                         szPlugin);
                     gUserManager.DropUser(u);
-          delete [] szPlugin;
-      return true;
-    }
-
-    packet >> nLongLen;
-        char* szMessage = new char[nLongLen+1];
+                    return true;
+                }
+
+                packet >> nLongLen;
+                char szMessage[nLongLen+1];
                 for (unsigned short i = 0; i < nLongLen; i++)
                     packet >> szMessage[i];
@@ -2852 +2841 @@
                 } // switch nICBMCommand
 
-        delete [] szPlugin;
-        delete [] szMessage;
-
-    break;
-      }
+                break;
+            }
 
 #ifdef USE_OPENSSL
@@ -3348 +3334 @@
               packet.incDataPosRead(4); //Unknown
               unsigned long nLen = packet.UnpackUnsignedLong();
-              char* szName = new char[nLen+1];
+              char szName[nLen+1];
               for (unsigned long i = 0; i < nLen; i++)
                 packet >> szName[i];
@@ -3354 +3340 @@
 
               nLen = packet.UnpackUnsignedLong();
-              char* szFullName = new char[nLen+1];
+              char szFullName[nLen+1];
               for (unsigned long i = 0; i < nLen; i++)
                 packet >> szFullName[i];
@@ -3363 +3349 @@
               gLog.Info("%s%s has %s (%s).\n", szInfo, u->GetAlias(), szName,
                                                szFullName);
-              delete [] szName;
-              delete [] szFullName;
             }
             break;
@@ -3372 +3356 @@
           {
             gLog.Info("%sPhone Book reply from %s.\n", szInfo, u->GetAlias());
-            PhoneBookEntry *pb = new PhoneBookEntry[nEntries];
+            struct PhoneBookEntry pb[nEntries];
             for (unsigned long i = 0; i < nEntries; i ++)
             {
@@ -3445 +3429 @@
             u->SetEnableSave(true);
             u->SavePhoneBookInfo();
-            delete [] pb;
 
             PushPluginSignal(new CICQSignal(SIGNAL_UPDATExUSER, USER_PHONExBOOK,
@@ -3472 +3455 @@
 
             unsigned long nLen = packet.UnpackUnsignedLong();
-            char* data = new char[nLen];
+            char data[nLen];
             for (unsigned long i = 0; i < nLen; i++)
             {
@@ -3479 +3462 @@
 
             write(nFD, data, nLen);
-            delete [] data;
 
             u->SetEnableSave(false);
@@ -3679 +3661 @@
             packet.incDataPosRead(4); //Unknown
             unsigned long nLen = packet.UnpackUnsignedLong();
-            char* szName = new char[nLen+1];
+            char szName[nLen+1];
             for (unsigned long i = 0; i < nLen; i++)
               packet >> szName[i];
@@ -3685 +3667 @@
 
             nLen = packet.UnpackUnsignedLong();
-            char* szFullName = new char[nLen+1];
+            char szFullName[nLen+1];
             for (unsigned long i = 0; i < nLen; i++)
               packet >> szFullName[i];
@@ -3694 +3676 @@
             gLog.Info("%s%s has %s (%s).\n", szInfo, u->GetAlias(), szName,
                                              szFullName);
-            delete [] szName;
-            delete [] szFullName;
           }
         }

trunk/licq/src/icqd.cpp

@@ -1043 +1043 @@
 
   size_t nLen = strlen(BASE_DIR) + sizeof(file) + sizeof(suffix) + 2;
-  char* szTmpName = new char[nLen];
-  char* szFilename = new char[nLen];
-  char buff[128];
+  char szTmpName[nLen], szFilename[nLen], buff[128];
   int nRet, n, fd;
  
@@ -1061 +1059 @@
       gLog.Error("%sFailed updating %s: `%s'\n", L_ERRORxSTR,
                  szFilename, strerror(errno));
-    delete [] szTmpName;
-    delete [] szFilename;
     return;
   }
@@ -1110 +1106 @@
     gLog.Error("%sFailed updating %s: `%s'\n", L_ERRORxSTR,
                szFilename, strerror(errno));
-
-  delete [] szTmpName;
-  delete [] szFilename;
 }
 
@@ -2513 +2506 @@
     if (!bIsAck)
     {
-      char* szFilename = new char[nFilenameLen+1];
+      char szFilename[nFilenameLen+1];
       for (unsigned short i = 0; i < nFilenameLen; i++)
         packet >> szFilename[i];
@@ -2526 +2519 @@
       nEventType = ON_EVENT_FILE;
       pEvent = e;
-
-      delete [] szFilename;
     }
     else
@@ -2623 +2614 @@
     packet.incDataPosRead(18);
     packet >> nLongLen; // plugin len
-      char* szPlugin = new char[nLongLen+1];
+    char szPlugin[nLongLen+1];
     for (unsigned long i = 0; i < nLongLen; i++)
       packet >> szPlugin[i];
@@ -2649 +2640 @@
 
     packet >> nLongLen;
-      char* szMessage = new char[nLongLen+1];
+    char szMessage[nLongLen+1];
     for (unsigned long i = 0; i < nLongLen; i++)
       packet >> szMessage[i];
@@ -2662 +2653 @@
     ProcessMessage(u, packet, msg, nCommand, nMask, nMsgID,
                    nSequence, bIsAck, bNewUser);
-      delete [] szPlugin;
-      delete [] szMessage;
-      return;
-
-      break; // bah!
-    }
+    return;
+
+    break; // bah!
+  }
 
   default:

trunk/licq/src/onevent.cpp

@@ -107 +107 @@
     if (strlen(szFullParam))
     {
-      char* szCmd = new char[strlen(m_szCommand) + strlen(szFullParam) + 8];
+      char szCmd[strlen(m_szCommand) + strlen(szFullParam) + 8];
       sprintf(szCmd, "%s %s &", m_szCommand, szFullParam);
       system(szCmd);
-      delete [] szCmd;
     }