Changeset 543

Show
Ignore:
Timestamp:
05/03/06 00:20:38 (3 years ago)
Author:
oxff
Message:

nepenthes-experimental.core: streamlined module single-string logging, checked for possible introduced format string vulns and fixed 3 log messages in bindshell handlers in shellcode-signature

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • nepenthes/branches/nepenthes-experimental/modules/dnsresolve-adns/dnsresolve-adns.cpp

    r408 r543  
    8888        g_Nepenthes->getDNSMgr()->registerDNSHandler(this); 
    8989        REG_EVENT_HANDLER(this); 
    90         logDebug("%s","adns_init() Success\n"); 
     90        logDebug("adns_init() Success\n"); 
    9191 
    9292        return true; 

  • nepenthes/branches/nepenthes-experimental/modules/download-csend/CSendDownloadHandler.cpp

    r332 r543  
    8080        if (m_Config == NULL) 
    8181        { 
    82                 logCrit("%s","I need a config\n"); 
     82                logCrit("I need a config\n"); 
    8383                return false; 
    8484        } 
    …… 
    9090        catch(...) 
    9191        { 
    92                 logCrit("%s","Error setting needed vars, check your config\n"); 
     92                logCrit("Error setting needed vars, check your config\n"); 
    9393                return false; 
    9494        } 

  • nepenthes/branches/nepenthes-experimental/modules/download-curl/download-curl.cpp

    r408 r543  
    8181        if ( m_Config == NULL ) 
    8282        { 
    83                 logCrit("%s","I need a config\n"); 
     83                logCrit("I need a config\n"); 
    8484                return false; 
    8585        } 
    …… 
    9292    } catch ( ... ) 
    9393        { 
    94                 logCrit("%s","Error setting needed vars, check your config\n"); 
     94                logCrit("Error setting needed vars, check your config\n"); 
    9595                return false; 
    9696        } 

  • nepenthes/branches/nepenthes-experimental/modules/download-ftp/CTRLDialogue.cpp

    r343 r543  
    115115        if (m_Download == NULL && m_State < FTP_RETR) 
    116116        { 
    117                 logWarn("%s","broken ftp daemon \n"); 
     117                logWarn("broken ftp daemon \n"); 
    118118                return CL_DROP; 
    119119        } 
    …… 
    207207                                if (strncmp((char *)m_Buffer->getData() + iStart,"150 ",4) == 0) 
    208208                                { 
    209                                         logDebug("%s","RETR accepted\n"); 
     209                                        logDebug("RETR accepted\n"); 
    210210                                }else 
    211211                                if (strncmp((char *)m_Buffer->getData() + iStart,"226 ",4) == 0) 
    212212                                { 
    213                                         logDebug("%s","Transferr finished\n"); 
     213                                        logDebug("Transferr finished\n"); 
    214214                                        sendQuit(); 
    215215                                        m_State = FTP_QUIT; 
    …… 
    353353        if (strncmp(msg,"230 ",4) == 0) 
    354354        { 
    355                 logDebug("%s","Pass accepted, logged in \n"); 
     355                logDebug("Pass accepted, logged in \n"); 
    356356                return true; 
    357357        }else 
    …… 
    376376        if (strncmp(msg,"200 ",4) == 0) 
    377377        { 
    378                 logDebug("%s","Type accepted \n"); 
     378                logDebug("Type accepted \n"); 
    379379                return true; 
    380380        }else 
    …… 
    388388void CTRLDialogue::sendPort() 
    389389{ 
    390         logDebug("%s","System ... \n"); 
     390        logDebug("System ... \n"); 
    391391 
    392392        uint32_t ip; 
    …… 
    468468        if (strncmp(msg,"200 ",4) == 0) 
    469469        { 
    470                 logDebug("%s","Port accepted\n"); 
     470                logDebug("Port accepted\n"); 
    471471                return true; 
    472472        }else 
    …… 
    492492        if (strncmp(msg,"150 ",4) == 0) 
    493493        { 
    494                 logDebug("%s","Retr accepted\n"); 
     494                logDebug("Retr accepted\n"); 
    495495                return true; 
    496496        }else 
    …… 
    514514        if (strncmp(msg,"221 ",4) == 0) 
    515515        { 
    516                 logDebug("%s","Quit accepted\n"); 
     516                logDebug("Quit accepted\n"); 
    517517                return true; 
    518518        }else 
    …… 
    535535        if (strncmp(msg,"250 ",4) == 0) 
    536536        { 
    537                 logDebug("%s","CWD accepted\n"); 
    538                 return true; 
    539         }else 
    540         { 
    541                 return false; 
    542         } 
    543 } 
    544  
     537                logDebug("CWD accepted\n"); 
     538                return true; 
     539        }else 
     540        { 
     541                return false; 
     542        } 
     543} 
     544 

  • nepenthes/branches/nepenthes-experimental/modules/download-ftp/FILEDialogue.cpp

    r332 r543  
    9494        if (m_Download == NULL) 
    9595        { 
    96                 logWarn("%s","broken ftp server connected 2 times, dropping second connection\n"); 
     96                logWarn("broken ftp server connected 2 times, dropping second connection\n"); 
    9797                return CL_DROP; 
    9898        } 

  • nepenthes/branches/nepenthes-experimental/modules/download-ftp/download-ftp.cpp

    r343 r543  
    123123        if ( m_Config == NULL ) 
    124124        { 
    125                 logCrit("%s","I need a config\n"); 
     125                logCrit("I need a config\n"); 
    126126                return false; 
    127127        } 
    …… 
    148148        } catch ( ... ) 
    149149        { 
    150                 logCrit("%s","Error setting needed vars, check your config\n"); 
     150                logCrit("Error setting needed vars, check your config\n"); 
    151151                return false; 
    152152        } 

  • nepenthes/branches/nepenthes-experimental/modules/download-link/download-link.cpp

    r332 r543  
    7878        if (m_Config == NULL) 
    7979        { 
    80                 logCrit("%s","I need a config\n"); 
     80                logCrit("I need a config\n"); 
    8181                return false; 
    8282        } 
    …… 
    8888        catch(...) 
    8989        { 
    90                 logCrit("%s","Error setting needed vars, check your config\n"); 
     90                logCrit("Error setting needed vars, check your config\n"); 
    9191                return false; 
    9292        } 

  • nepenthes/branches/nepenthes-experimental/modules/download-rcp/RCPDialogue.cpp

    r341 r543  
    123123        { 
    124124        case RCP_STATE_REQUEST: 
    125                 logSpam("%s","RCP STATE_REQUEST\n"); 
     125                logSpam("RCP STATE_REQUEST\n"); 
    126126                m_Buffer->add(msg->getMsg(),msg->getSize()); 
    127127                if (m_Buffer->getSize() == 1 && *(char *)m_Buffer->getData() == 0) 
    …… 
    140140        case RCP_STATE_FILESTATS: 
    141141                m_Buffer->add(msg->getMsg(),msg->getSize()); 
    142                 logSpam("%s","RCP STATE_FILESTATS\n"); 
     142                logSpam("RCP STATE_FILESTATS\n"); 
    143143                { 
    144144                        // "C0644 98 7819 foo.exe\n" 

  • nepenthes/branches/nepenthes-experimental/modules/download-rcp/download-rcp.cpp

    r341 r543  
    8080        if (m_Config == NULL) 
    8181        { 
    82                 logCrit("%s","I need a config\n"); 
     82                logCrit("I need a config\n"); 
    8383                return false; 
    8484        } 
    …… 
    9090        catch(...) 
    9191        { 
    92                 logCrit("%s","Error setting needed vars, check your config\n"); 
     92                logCrit("Error setting needed vars, check your config\n"); 
    9393                return false; 
    9494        } 

  • nepenthes/branches/nepenthes-experimental/modules/download-tftp/TFTPDownloadHandler.cpp

    r332 r543  
    8585        if ( m_Config == NULL ) 
    8686        { 
    87                 logCrit("%s","I need a config\n"); 
     87                logCrit("I need a config\n"); 
    8888                return false; 
    8989        } 
    …… 
    9595    } catch ( ... ) 
    9696        { 
    97                 logCrit("%s","Error setting needed vars, check your config\n"); 
     97                logCrit("Error setting needed vars, check your config\n"); 
    9898                return false; 
    9999        } 

  • nepenthes/branches/nepenthes-experimental/modules/log-download/log-download.cpp

    r321 r543  
    107107        if ( m_Config == NULL ) 
    108108         { 
    109                  logCrit("%s","I need a config\n"); 
     109                 logCrit("I need a config\n"); 
    110110                 return false; 
    111111         } 
    …… 
    130130         } catch ( ... ) 
    131131         { 
    132                  logCrit("%s","Error setting needed vars, check your config\n"); 
     132                 logCrit("Error setting needed vars, check your config\n"); 
    133133                 return false; 
    134134         } 
    …… 
    220220 
    221221        default: 
    222                 logWarn("%s","this should not happen\n"); 
     222                logWarn("this should not happen\n"); 
    223223        } 
    224224        return 0; 

  • nepenthes/branches/nepenthes-experimental/modules/log-irc/IrcDialogue.cpp

    r378 r543  
    136136                if ( ((socks4_header_t *)msg->getMsg())->ucCommand == 90 ) 
    137137                { 
    138                         logInfo("%s","connected to ircd via tor\n"); 
     138                        logInfo("connected to ircd via tor\n"); 
    139139                        m_State = IRCDIA_CONNECTED; 
    140140 
    …… 
    145145                } else 
    146146                { 
    147                         logWarn("%s","tor did not accept our connection \n"); 
     147                        logWarn("tor did not accept our connection \n"); 
    148148                        return CL_DROP; 
    149149                } 

  • nepenthes/branches/nepenthes-experimental/modules/log-irc/log-irc.cpp

    r502 r543  
    117117                if ( m_Config == NULL ) 
    118118                { 
    119                         logCrit("%s","I need a config\n"); 
     119                        logCrit("I need a config\n"); 
    120120                        return false; 
    121121                } 
    …… 
    141141                } catch ( ... ) 
    142142                { 
    143                         logCrit("%s","Error setting needed vars, check your config\n"); 
     143                        logCrit("Error setting needed vars, check your config\n"); 
    144144                        return false; 
    145145                } 

  • nepenthes/branches/nepenthes-experimental/modules/log-prelude/log-prelude.cpp

    r532 r543  
    135135        if ( m_Config == NULL ) 
    136136        { 
    137                 logCrit("%s","I need a config\n"); 
     137                logCrit("I need a config\n"); 
    138138                return false; 
    139139        } 
    …… 
    151151        } catch ( ... ) 
    152152        { 
    153                 logCrit("%s","Error setting needed vars, check your config\n"); 
     153                logCrit("Error setting needed vars, check your config\n"); 
    154154                return false; 
    155155        } 
    …… 
    236236        return true; 
    237237#else  
    238         logCrit("%s","Module log-prelude is compiled without libprelude, this wont work, reconfigure the whole source and recompile"); 
     238        logCrit("Module log-prelude is compiled without libprelude, this wont work, reconfigure the whole source and recompile"); 
    239239        return false; 
    240240#endif 
    …… 
    354354 
    355355        default: 
    356                 logWarn("%s","this should not happen\n"); 
     356                logWarn("this should not happen\n"); 
    357357        } 
    358358        return 0; 
    …… 
    364364     
    365365 
    366         logInfo("%s","LogPrelude EVENT EV_SOCK_TCP_ACCEPT\n"); 
     366        logInfo("LogPrelude EVENT EV_SOCK_TCP_ACCEPT\n"); 
    367367 
    368368#ifdef HAVE_LIBPRELUDE 
    …… 
    437437        } 
    438438 
    439         logInfo("%s","LogPrelude EVENT EV_SOCK_TCP_CLOSE\n"); 
     439        logInfo("LogPrelude EVENT EV_SOCK_TCP_CLOSE\n"); 
    440440 
    441441#ifdef HAVE_LIBPRELUDE 
    …… 
    499499void LogPrelude::handleShellcodeDone(Event *event) 
    500500{ 
    501         logInfo("%s", "LogPrelude EVENT EV_SHELLCODE_DONE\n"); 
     501        logInfo("LogPrelude EVENT EV_SHELLCODE_DONE\n"); 
    502502 
    503503#ifdef HAVE_LIBPRELUDE 
    …… 
    683683void LogPrelude::handleDialogueAssignAndDone(Event *event) 
    684684{ 
    685          logInfo("%s", "LogPrelude EVENT EV_ASSIGN_AND_DONE\n"); 
     685         logInfo("LogPrelude EVENT EV_ASSIGN_AND_DONE\n"); 
    686686 
    687687#ifdef HAVE_LIBPRELUDE 

  • nepenthes/branches/nepenthes-experimental/modules/log-surfnet/log-surfnet.cpp

    r341 r543  
    111111        if ( m_Config == NULL ) 
    112112        { 
    113                 logCrit("%s","I need a config\n"); 
     113                logCrit("I need a config\n"); 
    114114                return false; 
    115115        } 
    …… 
    130130        } catch ( ... ) 
    131131        { 
    132                 logCrit("%s","Error setting needed vars, check your config\n"); 
     132                logCrit("Error setting needed vars, check your config\n"); 
    133133                return false; 
    134134        } 
    …… 
    237237 
    238238        default: 
    239                 logWarn("%s","this should not happen\n"); 
     239                logWarn("this should not happen\n"); 
    240240        } 
    241241 
    …… 
    284284 
    285285        default: 
    286                 logWarn("%s","this should not happen\n"); 
     286                logWarn("this should not happen\n"); 
    287287        } 
    288288 
    …… 
    318318 
    319319                default: 
    320                         logWarn("%s","this should not happen\n"); 
     320                        logWarn("this should not happen\n"); 
    321321                } 
    322322        } 

  • nepenthes/branches/nepenthes-experimental/modules/module-bridge/module-bridge.cpp

    r343 r543  
    115115        if ( m_Config == NULL ) 
    116116        { 
    117                 logCrit("%s","I need a config\n"); 
     117                logCrit("I need a config\n"); 
    118118                return false; 
    119119        } 
    …… 
    130130        } catch ( ... ) 
    131131        { 
    132                 logCrit("%s","Error setting needed vars, check your config\n"); 
     132                logCrit("Error setting needed vars, check your config\n"); 
    133133                return false; 
    134134        } 

  • nepenthes/branches/nepenthes-experimental/modules/module-portwatch/module-portwatch.cpp

    r321 r543  
    108108        if ( m_Config == NULL ) 
    109109        { 
    110                 logCrit("%s","I need a config\n"); 
     110                logCrit("I need a config\n"); 
    111111                return false; 
    112112        } 
    …… 
    120120        } catch ( ... ) 
    121121        { 
    122                 logCrit("%s","Error setting needed vars, check your config\n"); 
     122                logCrit("Error setting needed vars, check your config\n"); 
    123123                return false; 
    124124        } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_bielefeld_connect.cpp

    r341 r543  
    136136                if ((diaf = g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory")) == NULL) 
    137137                { 
    138                         logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     138                        logCrit("No WinNTShell DialogueFactory availible \n"); 
    139139                        return SCH_DONE; 
    140140                } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_bind.cpp

    r341 r543  
    7373        } catch ( ... ) 
    7474        { 
    75                 logCrit("%s","Error setting needed vars, check your config\n"); 
     75                logCrit("Error setting needed vars, check your config\n"); 
    7676                return false; 
    7777        } 
    …… 
    156156                        if ( (socket = g_Nepenthes->getSocketMgr()->bindTCPSocket(0,port,60,30)) == NULL ) 
    157157                        { 
    158                                 logCrit("%s","Could not bind socket %u \n",port); 
     158                                logCrit("Could not bind socket %u\n", port); 
    159159                                return SCH_DONE; 
    160160                        } 
    …… 
    163163                        if ( (diaf = g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory")) == NULL ) 
    164164                        { 
    165                                 logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     165                                logCrit("No WinNTShell DialogueFactory availible \n"); 
    166166                                return SCH_DONE; 
    167167                        } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_cmd.cpp

    r341 r543  
    106106        if (g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory") == NULL) 
    107107                { 
    108                         logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     108                        logCrit("No WinNTShell DialogueFactory availible \n"); 
    109109                        return SCH_DONE; 
    110110                } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_connect.cpp

    r341 r543  
    7676        } catch ( ... ) 
    7777        { 
    78                 logCrit("%s","Error setting needed vars, check your config\n"); 
     78                logCrit("Error setting needed vars, check your config\n"); 
    7979                return false; 
    8080        } 
    …… 
    180180                        if ((diaf = g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory")) == NULL) 
    181181                        { 
    182                                 logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     182                                logCrit("No WinNTShell DialogueFactory availible \n"); 
    183183                                return SCH_DONE; 
    184184                        } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_connect_trans.cpp

    r341 r543  
    7979        } catch ( ... ) 
    8080        { 
    81                 logCrit("%s","Error setting needed vars, check your config\n"); 
     81                logCrit("Error setting needed vars, check your config\n"); 
    8282                return false; 
    8383        } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_createprocess.cpp

    r341 r543  
    128128                if (g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory") == NULL) 
    129129                { 
    130                         logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     130                        logCrit("No WinNTShell DialogueFactory availible \n"); 
    131131                        return SCH_DONE; 
    132132                } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_mainz_bind.cpp

    r341 r543  
    122122                if ((socket = g_Nepenthes->getSocketMgr()->bindTCPSocket(0,port,60,30)) == NULL) 
    123123                { 
    124                         logCrit("%s","Could not bind socket %u \n",port); 
     124                        logCrit("Could not bind socket %u\n", port); 
    125125                        return SCH_DONE; 
    126126                } 
    …… 
    129129                if ((diaf = g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory")) == NULL) 
    130130                { 
    131                         logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     131                        logCrit("No WinNTShell DialogueFactory availible \n"); 
    132132                        return SCH_DONE; 
    133133                } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_wget.cpp

    r372 r543  
    172172                        if (isprint(url[i]) == false) 
    173173                        { 
    174                                 logWarn("%s","wget url contained unprintable chars \n"); 
     174                                logWarn("wget url contained unprintable chars \n"); 
    175175                                return SCH_NOTHING; 
    176176                        } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/sch_generic_winexec.cpp

    r341 r543  
    104104                if (g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory") == NULL) 
    105105                { 
    106                         logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     106                        logCrit("No WinNTShell DialogueFactory availible \n"); 
    107107                        return SCH_DONE; 
    108108                } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-generic/shellcode-generic.cpp

    r500 r543  
    123123        if (m_Config == NULL) 
    124124        { 
    125                 logCrit("%s","I need a config\n"); 
     125                logCrit("I need a config\n"); 
    126126                return false; 
    127127        } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-signatures/sch_namespace_bindshell.cpp

    r445 r543  
    9595                if ( (socket = g_Nepenthes->getSocketMgr()->bindTCPSocket(0,port,60,30)) == NULL ) 
    9696                { 
    97                         logCrit("%s","Could not bind socket %u \n",port); 
     97                        logCrit("Could not bind socket %u\n", port); 
    9898                        return SCH_DONE; 
    9999                } 
    …… 
    102102                if ( (diaf = g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory")) == NULL ) 
    103103                { 
    104                         logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     104                        logCrit("No WinNTShell DialogueFactory availible \n"); 
    105105                        return SCH_DONE; 
    106106                } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-signatures/sch_namespace_connectbackshell.cpp

    r499 r543  
    164164                if ((diaf = g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory")) == NULL) 
    165165                { 
    166                         logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     166                        logCrit("No WinNTShell DialogueFactory availible \n"); 
    167167                        return SCH_DONE; 
    168168                } 

  • nepenthes/branches/nepenthes-experimental/modules/shellcode-signatures/sch_namespace_execute.cpp

    r499 r543  
    8181                 if (g_Nepenthes->getFactoryMgr()->getFactory("WinNTShell DialogueFactory") == NULL) 
    8282                 { 
    83                          logCrit("%s","No WinNTShell DialogueFactory availible \n"); 
     83                         logCrit("No WinNTShell DialogueFactory availible \n"); 
    8484                         return SCH_DONE; 
    8585                 } 

  • nepenthes/branches/nepenthes-experimental/modules