removing autobahn report. add required closing behavior to fragmented control messages

Author: jeremyhahn

src/cwebsocket/client.c

@@ -703,9 +703,12 @@ int cwebsocket_client_read_data(cwebsocket_client *websocket) {
 	}
 	else if(frame.opcode == CONTINUATION) {
 		syslog(LOG_DEBUG, "cwebsocket_client_read_data: received CONTINUATION opcode");
-		return -1; // continuations are accounted for in read loop
+		return 0;
 	}
 	else if(frame.opcode == PING) {
+		if(frame.fin == 0) {
+			cwebsocket_client_close(websocket, 1002, "control message must not be fragmented");
+		}
 		if(frame.payload_len > 125) {
 			cwebsocket_client_close(websocket, 1002, "control frames must not exceed 125 bytes");
 			return -1;

src/websocket-server.c

@@ -63,7 +63,7 @@ void server_print_program_usage(const char *progname) {
 	exit(0);
 }
 
-int server_main(int argc, char **argv) {
+int main(int argc, char **argv) {
 
 	server_print_program_header();
 

src/websocket-testsuite.c

@@ -78,13 +78,13 @@ cwebsocket_subprotocol* autobahn_testsuite_new() {
 	return protocol;
 }
 
-int testsuite_main_exit(int exit_status) {
+int main_exit(int exit_status) {
 	syslog(LOG_DEBUG, "exiting cwebsocket");
 	closelog();
 	return exit_status;
 }
 
-void testsuite_print_program_header() {
+void print_program_header() {
 	printf("\n");
 	printf("                      ______                    ______      _____ \n");
     printf(" _________      _________  /_______________________  /________  /_\n");
@@ -98,9 +98,9 @@ void testsuite_print_program_header() {
 	printf("\n");
 }
 
-int testsuite_main(int argc, char **argv) {
+int main(int argc, char **argv) {
 
-	testsuite_print_program_header();
+	print_program_header();
 
 	setlogmask(LOG_UPTO(LOG_DEBUG));
 	openlog("cwebsocket", LOG_CONS | LOG_PERROR, LOG_USER);

test/autobahn/fuzzingserver.json

@@ -5,7 +5,7 @@
    "outdir": "./reports/client",
    "webport": 8080,
 
-   "cases": ["*"],
+   "cases": ["5.3"],
    "exclude-cases": [],
    "exclude-agent-cases": {}
 }