[2.7] bpo-35194: Fix a wrong constant in cp932 codec. (GH-10420) (GH-10433)

Alexey Izbyshev · serhiy-storchaka · commit 0d165262d949 · 2018-11-10T07:47:12.000+02:00
This typo doesn't affect the result because wrong bits are discarded on implicit conversion to unsigned char, but it trips UBSan with -fsanitize=implicit-integer-truncation. (cherry picked from commit 7a69cf4) Co-authored-by: Alexey Izbyshev <izbyshev@ispras.ru>
diff --git a/Modules/cjkcodecs/_codecs_jp.c b/Modules/cjkcodecs/_codecs_jp.c
@@ -40,7 +40,7 @@ ENCODER(cp932)
             if (c == 0xf8f0)
                 OUT1(0xa0)
             else
-                OUT1(c - 0xfef1 + 0xfd)
+                OUT1(c - 0xf8f1 + 0xfd)
             NEXT(1, 1)
             continue;
         }

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ ENCODER(cp932)`
`40`	`40`	`if (c == 0xf8f0)`
`41`	`41`	`OUT1(0xa0)`
`42`	`42`	`else`
`43`		`- OUT1(c - 0xfef1 + 0xfd)`
	`43`	`+ OUT1(c - 0xf8f1 + 0xfd)`
`44`	`44`	`NEXT(1, 1)`
`45`	`45`	`continue;`
`46`	`46`	`}`