Possible Ring Buffer Fix #586

Merged
muellerr merged 20 commits from eive/fsfw:mueller/possible-ring-buffer-fix into development 2022-07-25 10:12:40 +02:00
Owner

Avoid memory leak if passed data is greater than maximum allowed size and overwriteOld iset set to true.

Avoid memory leak if passed data is greater than maximum allowed size and overwriteOld iset set to true.
muellerr added 2 commits 2022-03-24 16:47:29 +01:00
ring buffer cfg error warning
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
a891769a02
muellerr requested review from gaisser 2022-03-24 16:47:36 +01:00
muellerr added this to the v5.0.0 milestone 2022-03-24 16:47:43 +01:00
muellerr added the
bug
label 2022-03-24 16:47:48 +01:00
muellerr added 1 commit 2022-03-24 16:51:48 +01:00
add two additional tests for ring buffer
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
8a44c498c5
muellerr added 1 commit 2022-03-24 20:58:06 +01:00
added nullptr check
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
879223f38f
muellerr added 1 commit 2022-03-24 21:01:28 +01:00
some more tests fail
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
23f8e5cb41
gaisser added 1 commit 2022-03-25 15:21:56 +01:00
Fixed Unittests
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
ad57e6713e
gaisser added the
Breaking API Change
label 2022-03-25 15:23:37 +01:00
gaisser added 1 commit 2022-03-25 15:27:34 +01:00
Added changelog
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
70f575396d
Owner

Fixed Unittests

Fixed Unittests
gaisser refused to review 2022-03-25 15:30:34 +01:00
gaisser requested review from mohr 2022-03-25 15:30:39 +01:00
gaisser changed title from WIP: Possible Ring Buffer Fix to Possible Ring Buffer Fix 2022-03-25 18:38:06 +01:00
muellerr added 1 commit 2022-03-28 15:03:06 +02:00
Merge remote-tracking branch 'upstream/development' into mueller/possible-ring-buffer-fix
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
f598f7030f
muellerr added 1 commit 2022-03-28 15:07:58 +02:00
increase allowed read bytes
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
b7a316008a
muellerr added 1 commit 2022-04-11 17:35:59 +02:00
Merge branch 'development' into mueller/possible-ring-buffer-fix
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
7a5ce57bbc
muellerr added 5 commits 2022-04-25 15:12:01 +02:00
added missing empty implementation
Some checks are pending
fsfw/fsfw/pipeline/head Build started...
fsfw/fsfw/pipeline/pr-development This commit looks good
6aa54fe1d4
added related PR in changelog
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
7c2e50b665
Merge branch 'development' into mueller/small-stm32-hal-bugifx
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
b03a6684f9
Reviewed-on: #599
Merge remote-tracking branch 'upstream/development' into mueller/possible-ring-buffer-fix
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
b7ffd2653b
muellerr added 1 commit 2022-04-25 15:17:01 +02:00
Merge remote-tracking branch 'upstream/development' into mueller/possible-ring-buffer-fix
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
755dcf66a3
muellerr added 1 commit 2022-04-27 08:42:28 +02:00
Merge branch 'development' into mueller/possible-ring-buffer-fix
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
d3e7037759
Author
Owner

After talk about this PR: There will be an update to this PR where the function will actually cut off the tail of a packet which is too large and write the remaining packet size (maximum ring buffer size) into the ring buffer if overwriteOld is set to true.

After talk about this PR: There will be an update to this PR where the function will actually cut off the tail of a packet which is too large and write the remaining packet size (maximum ring buffer size) into the ring buffer if `overwriteOld` is set to true.
Owner

During implementation of the updated functionality for the overwriteOld flag, it showed that the overwriteOld flag might not be handled correctly in some cases.

Setting this PR to WIP until I figure it out.

During implementation of the updated functionality for the `overwriteOld` flag, it showed that the `overwriteOld` flag might not be handled correctly in some cases. Setting this PR to WIP until I figure it out.
mohr changed title from Possible Ring Buffer Fix to WIP: Possible Ring Buffer Fix 2022-05-06 10:39:36 +02:00
muellerr added 1 commit 2022-05-11 11:34:42 +02:00
Merge branch 'development' into mueller/possible-ring-buffer-fix
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
e1604b292c
muellerr added 1 commit 2022-05-13 11:30:36 +02:00
Merge remote-tracking branch 'upstream/development' into mueller/possible-ring-buffer-fix
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
ff40a71582
muellerr added 1 commit 2022-05-16 15:13:53 +02:00
Merge remote-tracking branch 'upstream/development' into mueller/possible-ring-buffer-fix
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
d871f55a89
muellerr added 20 commits 2022-06-21 10:48:06 +02:00
re-apply updated auto-formatter
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
7fee852dbd
cache the TCP client address now
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
e1dd27b9dd
remove commented code
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
13cda86d23
Reviewed-on: #628
apply afmt
All checks were successful
fsfw/fsfw/pipeline/head This commit looks good
d0fc360697
static STORE renamed
All checks were successful
fsfw/fsfw/pipeline/head This commit looks good
dac700b80a
clang-format
All checks were successful
fsfw/fsfw/pipeline/head This commit looks good
8e2597f609
bugfix for srv11
All checks were successful
fsfw/fsfw/pipeline/head This commit looks good
fsfw/fsfw/pipeline/pr-development This commit looks good
0b53b4873f
Merge branch 'development' into mueller/tcp-server-cache-client-addr
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
5e62258aa6
Reviewed-on: #630
Merge branch 'development' into mueller/tc-11-tweaks
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
f288d5120d
afmt
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
34658ef7db
added 3 new subservices
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
e60a665de4
added missing delete
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
f7cde80088
return end iterator instead of crashing
All checks were successful
fsfw/fsfw/pipeline/head This commit looks good
fsfw/fsfw/pipeline/pr-development This commit looks good
e59f1f26bf
Reviewed-on: #632
Merge branch 'development' into mueller/possible-fix-ccsds-distrib
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
7dddcdfd55
Reviewed-on: #633
Merge remote-tracking branch 'upstream/development' into mueller/possible-ring-buffer-fix
Some checks failed
fsfw/fsfw/pipeline/pr-development There was a failure building this commit
45792e86b9
muellerr added 1 commit 2022-06-21 10:48:57 +02:00
Merge remote-tracking branch 'upstream/development' into mueller/possible-ring-buffer-fix
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
2ebefdffae
muellerr added 1 commit 2022-07-18 14:38:06 +02:00
Merge branch 'development' into mueller/possible-ring-buffer-fix
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
4da18172b3
muellerr changed title from WIP: Possible Ring Buffer Fix to Possible Ring Buffer Fix 2022-07-20 09:26:49 +02:00
muellerr added 1 commit 2022-07-21 13:23:49 +02:00
Merge branch 'development' into mueller/possible-ring-buffer-fix
All checks were successful
fsfw/fsfw/pipeline/pr-development This commit looks good
9bbe1dc716
muellerr merged commit c96e304b68 into development 2022-07-25 10:12:40 +02:00
muellerr deleted branch mueller/possible-ring-buffer-fix 2022-07-25 10:12:42 +02:00
Sign in to join this conversation.
No description provided.