Răsfoiți Sursa

coroipcs_handler_dispatch(): Fix conn_info->service security value: -1 is not a good security value since it's equal to SOCKET_SERVICE_INIT

Signed-off-by: Jerome Flesch <jerome.flesch@netasq.com>
Reviewed-by: Jan Friesse <jfriesse@redhat.com>
Jerome Flesch 14 ani în urmă
părinte
comite
b112672115
2 a modificat fișierele cu 3 adăugiri și 2 ștergeri
  1. 1 1
      exec/coroipcs.c
  2. 2 1
      include/corosync/coroipc_ipc.h

+ 1 - 1
exec/coroipcs.c

@@ -1736,7 +1736,7 @@ int coroipcs_handler_dispatch (
 		 * the ipc connection
 		 * the ipc connection
 		 */
 		 */
 		if (conn_info->service == SOCKET_SERVICE_INIT) {
 		if (conn_info->service == SOCKET_SERVICE_INIT) {
-			conn_info->service = -1;
+			conn_info->service = SOCKET_SERVICE_SECURITY_VIOLATION;
 		}
 		}
 	} else
 	} else
 	if (revent & POLLIN) {
 	if (revent & POLLIN) {

+ 2 - 1
include/corosync/coroipc_ipc.h

@@ -157,7 +157,8 @@ struct coroipcs_zc_header {
 	uint64_t server_address;
 	uint64_t server_address;
 };
 };
 
 
-#define SOCKET_SERVICE_INIT	0xFFFFFFFF
+#define SOCKET_SERVICE_INIT					0xFFFFFFFF
+#define SOCKET_SERVICE_SECURITY_VIOLATION	0xFFFFFFFE
 
 
 #define ZC_ALLOC_HEADER		0xFFFFFFFF
 #define ZC_ALLOC_HEADER		0xFFFFFFFF
 #define ZC_FREE_HEADER		0xFFFFFFFE
 #define ZC_FREE_HEADER		0xFFFFFFFE