Skip to content

Full transaction messages, implement QoS and fix memory issues#1

Open
vsalvador wants to merge 34 commits into
IBM-IoT:masterfrom
vsalvador:vsalvador-full_transaction_cmds
Open

Full transaction messages, implement QoS and fix memory issues#1
vsalvador wants to merge 34 commits into
IBM-IoT:masterfrom
vsalvador:vsalvador-full_transaction_cmds

Conversation

@vsalvador

Copy link
Copy Markdown

The reversal algorithm of ENDXACT_PAYLOAD linked structures is not properly implemented. This patch fix the reversal algorithm and allow to retrive all index operations inside a single transaction.

vsalvador and others added 15 commits March 18, 2026 12:07
Updated README to show proper syntax
Implemented a proper linked list reversal for ENDXACT_PAYLOAD.
Allows to do what README says. if not explicit topic is defined, the table name is used as topic.
…tion_cmds

Merge branche to allow default tabname as topic
getMQTTServerInfo: len bytes are allocated, but then serverHost[len] = 0; is written. This causes an out-of-bounds write. It should be len + 1 bytes.

Protection for strlen crash
Functions that don't know the size of the buffer they receive. In your code, this affects `columnValueToString` and, by extension, `rowToCSV`.

am_open: set xact memory as PER_TRANSACTION and get memory pointer by name; similar to indexList;

`my_register_callback` is called on every DML operation (insert/update/delete).

Memory leak of net if MqttClient_Init fails
Readded fallback for MQTT topic to table name and added QoS setting.
Change example usage files
Reflect changes to README
Simplify linked list algorithm
@vsalvador vsalvador changed the title Vsalvador full transaction cmds Full transaction messages and lots of memory issues Mar 30, 2026
@vsalvador vsalvador changed the title Full transaction messages and lots of memory issues Full transaction messages, implement QoS and fix memory issues Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant