|
- /* TEMPLATE GENERATED TESTCASE FILE
- Filename: CWE124_Buffer_Underwrite__CWE839_connect_socket_02.c
- Label Definition File: CWE124_Buffer_Underwrite__CWE839.label.xml
- Template File: sources-sinks-02.tmpl.c
- */
- /*
- * @description
- * CWE: 124 Buffer Underwrite
- * BadSource: connect_socket Read data using a connect socket (client side)
- * GoodSource: Non-negative but less than 10
- * Sinks:
- * GoodSink: Ensure the array index is valid
- * BadSink : Improperly check the array index by not checking the lower bound
- * Flow Variant: 02 Control flow: if(1) and if(0)
- *
- * */
-
- #include "std_testcase.h"
-
- #ifdef _WIN32
- #include <winsock2.h>
- #include <windows.h>
- #include <direct.h>
- #pragma comment(lib, "ws2_32") /* include ws2_32.lib when linking */
- #define CLOSE_SOCKET closesocket
- #else /* NOT _WIN32 */
- #include <sys/types.h>
- #include <sys/socket.h>
- #include <netinet/in.h>
- #include <arpa/inet.h>
- #include <unistd.h>
- #define INVALID_SOCKET -1
- #define SOCKET_ERROR -1
- #define CLOSE_SOCKET close
- #define SOCKET int
- #endif
-
- #define TCP_PORT 27015
- #define IP_ADDRESS "127.0.0.1"
- #define CHAR_ARRAY_SIZE (3 * sizeof(data) + 2)
-
- #ifndef OMITBAD
-
- void CWE124_Buffer_Underwrite__CWE839_connect_socket_02_bad()
- {
- int data;
- /* Initialize data */
- data = -1;
- if(1)
- {
- {
- #ifdef _WIN32
- WSADATA wsaData;
- int wsaDataInit = 0;
- #endif
- int recvResult;
- struct sockaddr_in service;
- SOCKET connectSocket = INVALID_SOCKET;
- char inputBuffer[CHAR_ARRAY_SIZE];
- do
- {
- #ifdef _WIN32
- if (WSAStartup(MAKEWORD(2,2), &wsaData) != NO_ERROR)
- {
- break;
- }
- wsaDataInit = 1;
- #endif
- /* POTENTIAL FLAW: Read data using a connect socket */
- connectSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
- if (connectSocket == INVALID_SOCKET)
- {
- break;
- }
- memset(&service, 0, sizeof(service));
- service.sin_family = AF_INET;
- service.sin_addr.s_addr = inet_addr(IP_ADDRESS);
- service.sin_port = htons(TCP_PORT);
- if (connect(connectSocket, (struct sockaddr*)&service, sizeof(service)) == SOCKET_ERROR)
- {
- break;
- }
- /* Abort on error or the connection was closed, make sure to recv one
- * less char than is in the recv_buf in order to append a terminator */
- recvResult = recv(connectSocket, inputBuffer, CHAR_ARRAY_SIZE - 1, 0);
- if (recvResult == SOCKET_ERROR || recvResult == 0)
- {
- break;
- }
- /* NUL-terminate the string */
- inputBuffer[recvResult] = '\0';
- /* Convert to int */
- data = atoi(inputBuffer);
- }
- while (0);
- if (connectSocket != INVALID_SOCKET)
- {
- CLOSE_SOCKET(connectSocket);
- }
- #ifdef _WIN32
- if (wsaDataInit)
- {
- WSACleanup();
- }
- #endif
- }
- }
- if(1)
- {
- {
- int i;
- int buffer[10] = { 0 };
- /* POTENTIAL FLAW: Attempt to access a negative index of the array
- * This code does not check to see if the array index is negative */
- if (data < 10)
- {
- buffer[data] = 1;
- /* Print the array values */
- for(i = 0; i < 10; i++)
- {
- printIntLine(buffer[i]);
- }
- }
- else
- {
- printLine("ERROR: Array index is negative.");
- }
- }
- }
- }
-
- #endif /* OMITBAD */
-
- #ifndef OMITGOOD
-
- /* goodB2G1() - use badsource and goodsink by changing the second 1 to 0 */
- static void goodB2G1()
- {
- int data;
- /* Initialize data */
- data = -1;
- if(1)
- {
- {
- #ifdef _WIN32
- WSADATA wsaData;
- int wsaDataInit = 0;
- #endif
- int recvResult;
- struct sockaddr_in service;
- SOCKET connectSocket = INVALID_SOCKET;
- char inputBuffer[CHAR_ARRAY_SIZE];
- do
- {
- #ifdef _WIN32
- if (WSAStartup(MAKEWORD(2,2), &wsaData) != NO_ERROR)
- {
- break;
- }
- wsaDataInit = 1;
- #endif
- /* POTENTIAL FLAW: Read data using a connect socket */
- connectSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
- if (connectSocket == INVALID_SOCKET)
- {
- break;
- }
- memset(&service, 0, sizeof(service));
- service.sin_family = AF_INET;
- service.sin_addr.s_addr = inet_addr(IP_ADDRESS);
- service.sin_port = htons(TCP_PORT);
- if (connect(connectSocket, (struct sockaddr*)&service, sizeof(service)) == SOCKET_ERROR)
- {
- break;
- }
- /* Abort on error or the connection was closed, make sure to recv one
- * less char than is in the recv_buf in order to append a terminator */
- recvResult = recv(connectSocket, inputBuffer, CHAR_ARRAY_SIZE - 1, 0);
- if (recvResult == SOCKET_ERROR || recvResult == 0)
- {
- break;
- }
- /* NUL-terminate the string */
- inputBuffer[recvResult] = '\0';
- /* Convert to int */
- data = atoi(inputBuffer);
- }
- while (0);
- if (connectSocket != INVALID_SOCKET)
- {
- CLOSE_SOCKET(connectSocket);
- }
- #ifdef _WIN32
- if (wsaDataInit)
- {
- WSACleanup();
- }
- #endif
- }
- }
- if(0)
- {
- /* INCIDENTAL: CWE 561 Dead Code, the code below will never run */
- printLine("Benign, fixed string");
- }
- else
- {
- {
- int i;
- int buffer[10] = { 0 };
- /* FIX: Properly validate the array index and prevent a buffer underwrite */
- if (data >= 0 && data < (10))
- {
- buffer[data] = 1;
- /* Print the array values */
- for(i = 0; i < 10; i++)
- {
- printIntLine(buffer[i]);
- }
- }
- else
- {
- printLine("ERROR: Array index is out-of-bounds");
- }
- }
- }
- }
-
- /* goodB2G2() - use badsource and goodsink by reversing the blocks in the second if */
- static void goodB2G2()
- {
- int data;
- /* Initialize data */
- data = -1;
- if(1)
- {
- {
- #ifdef _WIN32
- WSADATA wsaData;
- int wsaDataInit = 0;
- #endif
- int recvResult;
- struct sockaddr_in service;
- SOCKET connectSocket = INVALID_SOCKET;
- char inputBuffer[CHAR_ARRAY_SIZE];
- do
- {
- #ifdef _WIN32
- if (WSAStartup(MAKEWORD(2,2), &wsaData) != NO_ERROR)
- {
- break;
- }
- wsaDataInit = 1;
- #endif
- /* POTENTIAL FLAW: Read data using a connect socket */
- connectSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
- if (connectSocket == INVALID_SOCKET)
- {
- break;
- }
- memset(&service, 0, sizeof(service));
- service.sin_family = AF_INET;
- service.sin_addr.s_addr = inet_addr(IP_ADDRESS);
- service.sin_port = htons(TCP_PORT);
- if (connect(connectSocket, (struct sockaddr*)&service, sizeof(service)) == SOCKET_ERROR)
- {
- break;
- }
- /* Abort on error or the connection was closed, make sure to recv one
- * less char than is in the recv_buf in order to append a terminator */
- recvResult = recv(connectSocket, inputBuffer, CHAR_ARRAY_SIZE - 1, 0);
- if (recvResult == SOCKET_ERROR || recvResult == 0)
- {
- break;
- }
- /* NUL-terminate the string */
- inputBuffer[recvResult] = '\0';
- /* Convert to int */
- data = atoi(inputBuffer);
- }
- while (0);
- if (connectSocket != INVALID_SOCKET)
- {
- CLOSE_SOCKET(connectSocket);
- }
- #ifdef _WIN32
- if (wsaDataInit)
- {
- WSACleanup();
- }
- #endif
- }
- }
- if(1)
- {
- {
- int i;
- int buffer[10] = { 0 };
- /* FIX: Properly validate the array index and prevent a buffer underwrite */
- if (data >= 0 && data < (10))
- {
- buffer[data] = 1;
- /* Print the array values */
- for(i = 0; i < 10; i++)
- {
- printIntLine(buffer[i]);
- }
- }
- else
- {
- printLine("ERROR: Array index is out-of-bounds");
- }
- }
- }
- }
-
- /* goodG2B1() - use goodsource and badsink by changing the first 1 to 0 */
- static void goodG2B1()
- {
- int data;
- /* Initialize data */
- data = -1;
- if(0)
- {
- /* INCIDENTAL: CWE 561 Dead Code, the code below will never run */
- printLine("Benign, fixed string");
- }
- else
- {
- /* FIX: Use a value greater than 0, but less than 10 to avoid attempting to
- * access an index of the array in the sink that is out-of-bounds */
- data = 7;
- }
- if(1)
- {
- {
- int i;
- int buffer[10] = { 0 };
- /* POTENTIAL FLAW: Attempt to access a negative index of the array
- * This code does not check to see if the array index is negative */
- if (data < 10)
- {
- buffer[data] = 1;
- /* Print the array values */
- for(i = 0; i < 10; i++)
- {
- printIntLine(buffer[i]);
- }
- }
- else
- {
- printLine("ERROR: Array index is negative.");
- }
- }
- }
- }
-
- /* goodG2B2() - use goodsource and badsink by reversing the blocks in the first if */
- static void goodG2B2()
- {
- int data;
- /* Initialize data */
- data = -1;
- if(1)
- {
- /* FIX: Use a value greater than 0, but less than 10 to avoid attempting to
- * access an index of the array in the sink that is out-of-bounds */
- data = 7;
- }
- if(1)
- {
- {
- int i;
- int buffer[10] = { 0 };
- /* POTENTIAL FLAW: Attempt to access a negative index of the array
- * This code does not check to see if the array index is negative */
- if (data < 10)
- {
- buffer[data] = 1;
- /* Print the array values */
- for(i = 0; i < 10; i++)
- {
- printIntLine(buffer[i]);
- }
- }
- else
- {
- printLine("ERROR: Array index is negative.");
- }
- }
- }
- }
-
- void CWE124_Buffer_Underwrite__CWE839_connect_socket_02_good()
- {
- goodB2G1();
- goodB2G2();
- goodG2B1();
- goodG2B2();
- }
-
- #endif /* OMITGOOD */
-
- /* Below is the main(). It is only used when building this testcase on
- its own for testing or for building a binary to use in testing binary
- analysis tools. It is not used when compiling all the testcases as one
- application, which is how source code analysis tools are tested. */
-
- #ifdef INCLUDEMAIN
-
- int main(int argc, char * argv[])
- {
- /* seed randomness */
- srand( (unsigned)time(NULL) );
- #ifndef OMITGOOD
- printLine("Calling good()...");
- CWE124_Buffer_Underwrite__CWE839_connect_socket_02_good();
- printLine("Finished good()");
- #endif /* OMITGOOD */
- #ifndef OMITBAD
- printLine("Calling bad()...");
- CWE124_Buffer_Underwrite__CWE839_connect_socket_02_bad();
- printLine("Finished bad()");
- #endif /* OMITBAD */
- return 0;
- }
-
- #endif
|