Maxim Korotkov

Several bugs are detected by SAST tools

2

Hi guys! Static analysis tools detected a some potential weakness in your project * 1 ``` QXlsx/WebServer/recurse.hpp in line 526: Access of moved variable 'prev'. ``` * 2 ``` This...

added checking of ptr before it was dereferenced

1

Hi! I found potential bug, currentWorksheet() can return nullptr, it follows that the pointer must be checked before it was uses

The result of FindWorkerNode() is usually checked against NULL.

In the function TaskConcurrentCancelCheck() the pointer "task" was utilized after checking against NULL, which can lead to dereference of the null pointer. To avoid the problem, added a separate handling...

src/backend/distributed/utils/background_jobs.c In the function static TaskExecutionStatus TaskConcurrentCancelCheck() ``` if (!task || task->status == BACKGROUND_TASK_STATUS_CANCELLING) \\ entering the condition if the task value is NULL { ... ereport(LOG, (errmsg( "task jobid/taskid...

The pointer was unsafely used after null checking

1

src/proto.c - bool send_startup_packet() ``` client = first_socket(&pool->waiting_client_list); if (client && client->replication) { server->replication = client->replication; ... } if (server->replication) { client->link = server; // client was dereferenced if server->replication...