[openwrt/openwrt] firmware-utils: fix mistake and improve logic in nec-enc

LEDE Commits lede-commits at lists.infradead.org
Thu Nov 12 12:22:35 EST 2020


ynezz pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/65f3e7ce1f4ad0abda9f0b98ffb7972960602b41

commit 65f3e7ce1f4ad0abda9f0b98ffb7972960602b41
Author: INAGAKI Hiroshi <musashino.open at gmail.com>
AuthorDate: Sat Oct 31 21:45:05 2020 +0900

    firmware-utils: fix mistake and improve logic in nec-enc
    
    this patch fixes/improves follows:
    
    - PATTERN_LEN is defined as a macro but unused
    - redundant logic in count-up for "ptn"
    
    Signed-off-by: INAGAKI Hiroshi <musashino.open at gmail.com>
---
 tools/firmware-utils/Makefile      | 2 +-
 tools/firmware-utils/src/nec-enc.c | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tools/firmware-utils/Makefile b/tools/firmware-utils/Makefile
index 7f1754a347..81c62d977a 100644
--- a/tools/firmware-utils/Makefile
+++ b/tools/firmware-utils/Makefile
@@ -7,7 +7,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME := firmware-utils
-PKG_RELEASE := 4
+PKG_RELEASE := 5
 
 include $(INCLUDE_DIR)/host-build.mk
 include $(INCLUDE_DIR)/kernel.mk
diff --git a/tools/firmware-utils/src/nec-enc.c b/tools/firmware-utils/src/nec-enc.c
index 3c4e38721e..a2be378586 100644
--- a/tools/firmware-utils/src/nec-enc.c
+++ b/tools/firmware-utils/src/nec-enc.c
@@ -47,7 +47,7 @@ static unsigned char buf_pattern[4096], buf[4096];
 
 int main(int argc, char **argv)
 {
-	int k_off = 0, ptn = 0, c, ret = EXIT_SUCCESS;
+	int k_off = 0, ptn = 1, c, ret = EXIT_SUCCESS;
 	char *ifn = NULL, *ofn = NULL, *key = NULL;
 	size_t n, k_len;
 	FILE *out, *in;
@@ -99,11 +99,11 @@ int main(int argc, char **argv)
 
 	while ((n = fread(buf, 1, sizeof(buf), in)) > 0) {
 		for (int i = 0; i < n; i++) {
-			buf_pattern[i] = ptn + 1;
+			buf_pattern[i] = ptn;
 			ptn++;
 
-			if (ptn > 250)
-				ptn = 0;
+			if (ptn > PATTERN_LEN)
+				ptn = 1;
 		}
 
 		k_off = xor_pattern(buf_pattern, n, key, k_len, k_off);



More information about the lede-commits mailing list