Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: one more Heap-Leak @Traceroute #5317

Closed
karamo opened this issue Nov 11, 2024 · 10 comments
Closed

[Bug]: one more Heap-Leak @Traceroute #5317

karamo opened this issue Nov 11, 2024 · 10 comments
Labels
bug Something isn't working

Comments

@karamo
Copy link

karamo commented Nov 11, 2024

Category

Other

Hardware

Not Applicable

Firmware Version

2.5.12 + fix

Description

ther is one more Heap-Leak @traceroute (red signs)
leakTR

Scenario:
FW 2.5.12 with fix PR #5311
3 T-Beam local direct with noch connection to other nodes.

Relevant log output

No response

@karamo karamo added the bug Something isn't working label Nov 11, 2024
@GUVWAF
Copy link
Member

GUVWAF commented Nov 12, 2024

Is this when sending or receiving traceroutes? Also, might it be it also happens on other packet types like position requests or just text messages?

@karamo
Copy link
Author

karamo commented Nov 12, 2024

I made Traceroute with MeshSense.
Other actions not tested.

@karamo
Copy link
Author

karamo commented Nov 12, 2024

At starting Meshsense and connecting via IP to that node, there is a down-peak followd by a jump up of the Heap.
MeshSense_Start

Log of Meshsense:
MeshSense_Start_IP_4Nodes.zip

Scenario: 4 T-Beam local direkt active

@karamo
Copy link
Author

karamo commented Nov 12, 2024

At he 1st DM there is a peak down, followed by up an no permanent Heap-down.
MEshSense_SendMsg
Log: MEshSense_SendMsg.zip

@karamo
Copy link
Author

karamo commented Nov 12, 2024

Sending Traceroute via MeshSense every 30s (FW limit), Heap permanently damaged:
MeshSense_Send-Traceroute

MeshSense Log: MeshSense_Send-Traceroute.zip

@karamo
Copy link
Author

karamo commented Nov 12, 2024

It seems to me that TraceRoute is interrupted if another packet is received in the meantime.

@karamo
Copy link
Author

karamo commented Nov 12, 2024

after ≈10 mins Heap partially recovered
MeshSense_Heappartrecover

@GUVWAF
Copy link
Member

GUVWAF commented Nov 12, 2024

@karamo Many thanks for all the testing.

Can you try if #5328 fixes it?

@karamo
Copy link
Author

karamo commented Nov 12, 2024

Can you try if #5328 fixes it?

Yes, ist running ... wait 1 to 3 hours ...

@karamo
Copy link
Author

karamo commented Nov 12, 2024

ok, it looks good.
Reboot 2.5.12 + Fix1 + Fix2
Significant points:
picture 1:
19:36:49 - 52428 B - Reboot and heap down with first slope to
19:56:31 - 38424 B - begin 2nd slope to jump up at
20:42:56 - 35884 B ^^
21:17:01 - 40220 B - start of MeshSense (red 1 in 2nd picture)
picture 2:
27:17:41 - 47724 B - maximum following slow down
21:34:43 - 45044 B - start 1st Traceroute every ≈30s no significant Heap-Leak (red 2 in 2nd picture)

MeshSense_IPconnect
MeshSense_Traceroute

Serial_Log: MemoAllInput_12.11.2024_21_51_13.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants