pespin submitted this change.
rlcmac: tbf_ul: Fix wrong field assignment (no bug in practice)
Fixes following warning detected by clang:
"""
tbf_ul.c:1118:37: warning: implicit conversion from enumeration type
'enum gprs_rlcmac_rlc_egprs_ul_reseg_bsn_state' to different enumeration type
'enum gprs_rlcmac_rlc_egprs_dl_reseg_bsn_state' [-Wenum-conversion]
blk->spb_status.block_status_dl = reseg_status;
"""
In practice it's not much of a problem since both fields are put
together in a union, so it ends up in the same place in memory.
Change-Id: Iccd57b74640754f2aebf81a149a633f141c6e38f
---
M src/rlcmac/tbf_ul.c
1 file changed, 21 insertions(+), 1 deletion(-)
diff --git a/src/rlcmac/tbf_ul.c b/src/rlcmac/tbf_ul.c
index 3786b30..d96891f 100644
--- a/src/rlcmac/tbf_ul.c
+++ b/src/rlcmac/tbf_ul.c
@@ -1115,7 +1115,7 @@
* function returns the reseg status
*/
reseg_status = egprs_ul_get_data(ul_tbf, bsn, &block_data);
- blk->spb_status.block_status_dl = reseg_status;
+ blk->spb_status.block_status_ul = reseg_status;
/*
* If it is first segment of the split block set the state of
To view, visit change 34616. To unsubscribe, or for help writing mail filters, visit settings.