Fix printf warnings for cran

jeroen · jeroen · commit 86e50671dc5c · 2023-12-04T13:36:50.000+01:00
diff --git a/NEWS b/NEWS
@@ -1,5 +1,6 @@
 1.8.8
   - Apply libyajl patches for CVE-2022-24795, CVE-2022-24795, CVE-2023-33460
+  - Fix printf warnings for cran
 
 1.8.7
   - toJSON(digits = NA) once again prints 15 digits max
diff --git a/src/parse.c b/src/parse.c
@@ -35,7 +35,7 @@ SEXP R_parse(SEXP x, SEXP bigint_as_char) {
 
     /* parser error */
     if (!node) {
-      Rf_errorcall(R_NilValue, errbuf);
+      Rf_errorcall(R_NilValue, "%s", errbuf);
     }
     SEXP out = ParseValue(node, bigint);
     yajl_tree_free(node);
diff --git a/src/push_parser.c b/src/push_parser.c
@@ -68,5 +68,5 @@ SEXP R_parse_connection(SEXP sConn, SEXP bigint_as_char){
     strncpy(errbuf, (char *) errstr, bufsize - 1);
     yajl_free_error(push_parser, errstr);
     yajl_free(push_parser);
-    Rf_error(errbuf);
+    Rf_error("%s", errbuf);
 }

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ SEXP R_parse(SEXP x, SEXP bigint_as_char) {`
`35`	`35`
`36`	`36`	`/* parser error */`
`37`	`37`	`if (!node) {`
`38`		`- Rf_errorcall(R_NilValue, errbuf);`
	`38`	`+ Rf_errorcall(R_NilValue, "%s", errbuf);`
`39`	`39`	`}`
`40`	`40`	`SEXP out = ParseValue(node, bigint);`
`41`	`41`	`yajl_tree_free(node);`
Original file line number	Diff line number	Diff line change
`@@ -68,5 +68,5 @@ SEXP R_parse_connection(SEXP sConn, SEXP bigint_as_char){`
`68`	`68`	`strncpy(errbuf, (char *) errstr, bufsize - 1);`
`69`	`69`	`yajl_free_error(push_parser, errstr);`
`70`	`70`	`yajl_free(push_parser);`
`71`		`- Rf_error(errbuf);`
	`71`	`+ Rf_error("%s", errbuf);`
`72`	`72`	`}`