Breaking update

CMakeLists.txt

@@ -5,6 +5,9 @@ set(CMAKE_CXX_STANDARD 20)
 
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++20 -Wall")
 
+add_compile_options(-fsanitize=address)
+add_link_options(-fsanitize=address)
+
 include_directories(
   include
 )

doc/multithread_maze_server.md

@@ -7,7 +7,7 @@ UML-диаграмма протокола пользовательского у
 ![UML-диаграмма протокола пользовательского уровня](maze.svg)
 
 ## Описание работы приложения
-Сервер `MazeServer` ожидает подключения клиентов по протоколу TCP на заданный порт (по умолчанию 1024). После установления соединения сервер запрашивает имя игрока, а затем начинает игру в лабиринте. Лабиринт генерируется случайным образом при старте каждой новой игровой сессии. Гарантируется наличие хотя бы одного пути из начальной точки `(0, 0)` в конечную точку `(2, 2)`. Добавление дополнительных стен происходит с соблюдением условия достижимости цели.
+Сервер `MazeServer` ожидает подключения клиентов по протоколу TCP на заданный порт (по умолчанию 2000). После установления соединения сервер запрашивает имя игрока, а затем начинает игру в лабиринте. Лабиринт генерируется случайным образом при старте каждой новой игровой сессии. Гарантируется наличие хотя бы одного пути из начальной точки `(0, 0)` в конечную точку `(2, 2)`. Добавление дополнительных стен происходит с соблюдением условия достижимости цели.
 
 Cхема и нумерация клеток лабиринта представлена на рисунке ниже.
 
@@ -25,9 +25,9 @@ Cхема и нумерация клеток лабиринта представ
 - `-s` — включает сервисный режим (без ограничений по количеству шагов).
      
 ```bash
-./maze-server -h localhost -p 1024 -n 10 -s
+./maze-server -h localhost -p 2000 -n 10 -s
 ```
-По умолчанию сокет сервера связывается с адресом `localhost:1024`, игрокам разрешено 10 шагов, сервисный режим выключен.
+По умолчанию сокет сервера связывается с адресом `localhost:2000`, игрокам разрешено 10 шагов, сервисный режим выключен.
 
 Завершение работы сервера — `Ctrl+C`.
 
@@ -48,7 +48,7 @@ Cхема и нумерация клеток лабиринта представ
 
 - `-p` — задаёт номер порта сервера.
 ```bash
-./maze_client -h localhost -p 1024
+./maze_client -h localhost -p 2000
 ```
-По умолчанию клиент пытается подключиться к адресу `localhost:1024`.
+По умолчанию клиент пытается подключиться к адресу `localhost:2000`.
 

include/client.hpp

@@ -13,6 +13,9 @@
 #include <unistd.h>
 #include <netdb.h>
 
+/*! Размер буфера для обмена данными между сервером и клиентом. */
+const int BUFFER_SIZE = 1024;
+
 /*! Класс клиента для взаимодействия с сервером игры в лабиринт. */
 class Client{
 private:

src/client/client.cpp

@@ -49,6 +49,10 @@ void Client::run(const std::string& h, const unsigned short p) {
     std::string player_name;
     std::getline(std::cin, player_name);
 
+    if (player_name.empty() || player_name.size() > BUFFER_SIZE) {
+        throw std::runtime_error("Incorrect name!");
+    }
+
     if (!ping(serv_addr)) {
         throw std::runtime_error("Connection lost!");
     }
@@ -61,25 +65,30 @@ void Client::game(){
     std::cout << "Игра началась!" << std::endl;
     //print_instructions();
 
-    char buffer[1024] = {0};
+    char buffer[BUFFER_SIZE] = {0};
     while (true) {
         std::cout << "Введите команду: ";
         std::string command;
         std::getline(std::cin, command);
 
+        if (command.empty() || command.size() > BUFFER_SIZE) {
+            std::cout << "Некорректная команда" << std::endl;
+            continue;
+        }
+
         // Отправка команды на сервер
         send(sock, command.c_str(), command.size(), 0);
 
         // Получение ответа от сервера
         memset(buffer, 0, sizeof(buffer));
         int bytes_received = recv(sock, buffer, sizeof(buffer), 0);
-        if (bytes_received <= 0) {
+        if (bytes_received == 0 || bytes_received > BUFFER_SIZE) {
             throw std::runtime_error("Connection lost!");
             //break;
         }
 
         std::string response(buffer);
-        std::cout << "Ответ сервера: " << response;
+        std::cout << "Ответ сервера: " << response << std::endl;
 
         // Проверка завершения игры
         if (response.find("вы выиграли") != std::string::npos ||

src/client/client_main.cpp

@@ -28,7 +28,7 @@
 int main(int argc,  char** argv){
     int opt;
     std::string host = "localhost";
-    short unsigned port = 1024u;
+    short unsigned port = 2000u;
 
     while ((opt = getopt(argc, argv, "h:p:")) != -1) {
         switch (opt) {

src/server/server.cpp

@@ -41,7 +41,7 @@ void Server::handle_client(int client_socket, bool mode, int steps) {
     std::string response;
 
     int bytes_received = recv(client_socket, buffer, sizeof(buffer), 0);
-    if (bytes_received <= 0) {
+    if (bytes_received <= 0 || bytes_received > BUFFER_SIZE) {
         std::cout << "Error in getting player name" << std::endl;;
         close(client_socket);
         return;
@@ -55,7 +55,7 @@ void Server::handle_client(int client_socket, bool mode, int steps) {
         memset(buffer, 0, sizeof(buffer));
         bytes_received = recv(client_socket, buffer, sizeof(buffer), 0);
 
-        if (bytes_received <= 0) {
+        if (bytes_received <= 0 || bytes_received > BUFFER_SIZE) {
             break;
         }
 
@@ -65,7 +65,7 @@ void Server::handle_client(int client_socket, bool mode, int steps) {
         int direction = -1;
         int new_position = current_position;
 
-        if (command == "вперёд") {
+        if (command == "вперед") {
             direction = 0;
             new_position = current_position + 3;
         } else if (command == "направо") {
@@ -94,14 +94,14 @@ void Server::handle_client(int client_socket, bool mode, int steps) {
             maze.set_moves_left(moves_left - 1);
             if (!check_status(maze))
                 break;
-            response = "там стена, осталось " + std::to_string(maze.get_moves_left()) + " ходов\n";
+            response = "там стена, осталось " + std::to_string(maze.get_moves_left()) + " ходов";
             send(client_socket, response.c_str(), response.size(), 0);
             continue;
         } else if (maze.is_wall(current_position, direction)) {
             maze.set_moves_left(moves_left - 1);
             if (!check_status(maze))
                 break;
-            response = "там стена, осталось " + std::to_string(maze.get_moves_left()) + " ходов\n";
+            response = "там стена, осталось " + std::to_string(maze.get_moves_left()) + " ходов";
             send(client_socket, response.c_str(), response.size(), 0);
             continue;
         } else {
@@ -119,7 +119,7 @@ void Server::handle_client(int client_socket, bool mode, int steps) {
 
             int x = current_position % 3;
             int y = current_position / 3;
-            std::string text("(" + std::to_string(x) + ", " + std::to_string(y) + ")\n");
+            std::string text("(" + std::to_string(x) + ", " + std::to_string(y) + ")");
             response = "успешно, осталось " + std::to_string(maze.get_moves_left()) + " ходов. Вы находитесь в " + text;
             send(client_socket, response.c_str(), response.size(), 0);
         }

src/server/server_main.cpp

@@ -30,7 +30,7 @@ int main(int argc, char **argv) {
     std::string host = "localhost";
     int steps = 10;
     bool service_mode = false;
-    short unsigned port = 1024u;
+    short unsigned port = 2000u;
 
     while ((opt = getopt(argc, argv, "h:p:sn:")) != -1) {
         switch (opt) {