Merge pull request #416 from arkedge/fix_return_in_PH_analyze_tlm_packet

meltingrabbit · web-flow · commit 3d07567fcfd5 · 2025-02-25T19:44:46.000+09:00
PH_analyze_tlm_packet の返り値を修正
diff --git a/tlm_cmd/packet_handler.c b/tlm_cmd/packet_handler.c
@@ -263,7 +263,7 @@ static PH_ACK PH_add_block_cmd_(const CommonCmdPacket* packet)
 PH_ACK PH_analyze_tlm_packet(const CommonTlmPacket* packet)
 {
   ctp_dest_flags_t flags;
-  if (!CTP_is_valid_packet(packet)) return PH_ACK_UNKNOWN;    // FIXME: 返り値変えたい
+  if (!CTP_is_valid_packet(packet)) return PH_ACK_INVALID_PACKET;
 
   flags = CTP_get_dest_flags(packet);
 

Original file line number	Diff line number	Diff line change
`@@ -263,7 +263,7 @@ static PH_ACK PH_add_block_cmd_(const CommonCmdPacket* packet)`
`263`	`263`	`PH_ACK PH_analyze_tlm_packet(const CommonTlmPacket* packet)`
`264`	`264`	`{`
`265`	`265`	`ctp_dest_flags_t flags;`
`266`		`- if (!CTP_is_valid_packet(packet)) return PH_ACK_UNKNOWN; // FIXME: 返り値変えたい`
	`266`	`+ if (!CTP_is_valid_packet(packet)) return PH_ACK_INVALID_PACKET;`
`267`	`267`
`268`	`268`	`flags = CTP_get_dest_flags(packet);`
`269`	`269`