while studying the code of the Reed-Solomon decoder, I came across a bug:
in line 208 of http://itpp.sourceforge.net/current/reedsolomon_8cpp_source.ht
ml the
syndrome should be calculated from variable cx and not rx. The syndrome of the
received block rx has been already calculated in l. 142 and rx has never been
changed ever since. But in l. 204 cx was calculated as the corrected (rx +
error ex) block and the following check of l. 208 should check if the
corrected one is a valid code word - and that is on cx, and not rx.
The bug fix will be part of the systematic decoding patch already.
Please confirm.
Regards Stephan
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Hi all,
while studying the code of the Reed-Solomon decoder, I came across a bug:
in line 208 of http://itpp.sourceforge.net/current/reedsolomon_8cpp_source.ht
ml the
syndrome should be calculated from variable cx and not rx. The syndrome of the
received block rx has been already calculated in l. 142 and rx has never been
changed ever since. But in l. 204 cx was calculated as the corrected (rx +
error ex) block and the following check of l. 208 should check if the
corrected one is a valid code word - and that is on cx, and not rx.
The bug fix will be part of the systematic decoding patch already.
Please confirm.
Regards Stephan
fyi bug is fixed in SVn rev. 1888