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

Remove All Packet command causes to_lab to break #71

Closed
jhroach14 opened this issue Dec 11, 2020 · 0 comments · Fixed by #74 or #75
Closed

Remove All Packet command causes to_lab to break #71

jhroach14 opened this issue Dec 11, 2020 · 0 comments · Fixed by #74 or #75
Assignees
Labels
bug Something isn't working
Milestone

Comments

@jhroach14
Copy link

Describe the bug
Sending the TO_REMOVE_ALL_PKT command to to_lab causes to_lab to unsubscribe from its own Message ID.
This means that to_lab will no longer respond to any commands and has to be restarted for it to function properly again. We ran into this problem at NASA Ames and figured that it was not intended behavior.

When testing for this bug be aware that it can be semi obscured by the fact that any commands for TO_LAB already on the Software Bus will still be processed by TO_Lab. This means that whether or not commands to TO_LAB sent after a TO_REMOVE_ALL_PKT command succeed is highly timing dependent.

To Reproduce
Steps to reproduce the behavior:

  1. Send TO_REMOVE_PKT command to TO_LAB
  2. Send any other command to TO_LAB
  3. Observe that the command has no effect

Expected behavior
You should not be able to send TO_LAB a command that puts it into a non functional state

Code snips
//This line in to_lab_app.c line 543 is the source of the problem
status = CFE_SB_Unsubscribe(TO_LAB_CMD_MID, TO_LAB_Global.Cmd_pipe);

System observed on:
This bug is independent of the environment cfs is run in

Reporter Info
James Roach - NASA Research Scientist
james.h.roach@nasa.gov

@skliper skliper self-assigned this Dec 14, 2020
@skliper skliper added the bug Something isn't working label Dec 14, 2020
@skliper skliper added this to the 2.5.0 milestone Dec 14, 2020
skliper added a commit to skliper/to_lab that referenced this issue Dec 14, 2020
Remove all was also removing subscriptions
from the command pipe, which leaves TO
inoperable.  The remove all command now
just removes all subscriptions to the Tlm_pipe
astrogeco added a commit that referenced this issue Dec 18, 2020
Fix #71, Remove all only from telemetry pipe
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
2 participants