[PATCH uci 1/4] file: use size_t for position and pointer
Hauke Mehrtens
hauke at hauke-m.de
Sat Oct 3 13:21:55 EDT 2020
The bufsz variable is used to store the size of the buf memory region
and pos is used to index a position in this memory. Use size_t for these
variables in the internal handling instaed of int to not break with big
files.
Signed-off-by: Hauke Mehrtens <hauke at hauke-m.de>
---
file.c | 14 +++++++-------
libuci.c | 2 +-
uci_internal.h | 8 ++++----
3 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/file.c b/file.c
index 5502a42..003f6c6 100644
--- a/file.c
+++ b/file.c
@@ -42,11 +42,11 @@
/*
* Fetch a new line from the input stream and resize buffer if necessary
*/
-__private void uci_getln(struct uci_context *ctx, int offset)
+__private void uci_getln(struct uci_context *ctx, size_t offset)
{
struct uci_parse_context *pctx = ctx->pctx;
char *p;
- int ofs;
+ size_t ofs;
if (pctx->buf == NULL) {
pctx->buf = uci_malloc(ctx, LINEBUF);
@@ -116,7 +116,7 @@ static void skip_whitespace(struct uci_context *ctx)
pctx->pos += 1;
}
-static inline void addc(struct uci_context *ctx, int *pos_dest, int *pos_src)
+static inline void addc(struct uci_context *ctx, size_t *pos_dest, size_t *pos_src)
{
struct uci_parse_context *pctx = ctx->pctx;
@@ -128,7 +128,7 @@ static inline void addc(struct uci_context *ctx, int *pos_dest, int *pos_src)
/*
* parse a double quoted string argument from the command line
*/
-static void parse_double_quote(struct uci_context *ctx, int *target)
+static void parse_double_quote(struct uci_context *ctx, size_t *target)
{
struct uci_parse_context *pctx = ctx->pctx;
char c;
@@ -163,7 +163,7 @@ static void parse_double_quote(struct uci_context *ctx, int *target)
/*
* parse a single quoted string argument from the command line
*/
-static void parse_single_quote(struct uci_context *ctx, int *target)
+static void parse_single_quote(struct uci_context *ctx, size_t *target)
{
struct uci_parse_context *pctx = ctx->pctx;
char c;
@@ -192,7 +192,7 @@ static void parse_single_quote(struct uci_context *ctx, int *target)
/*
* parse a string from the command line and detect the quoting style
*/
-static void parse_str(struct uci_context *ctx, int *target)
+static void parse_str(struct uci_context *ctx, size_t *target)
{
struct uci_parse_context *pctx = ctx->pctx;
bool next = true;
@@ -241,7 +241,7 @@ done:
static int next_arg(struct uci_context *ctx, bool required, bool name, bool package)
{
struct uci_parse_context *pctx = ctx->pctx;
- int val, ptr;
+ size_t val, ptr;
skip_whitespace(ctx);
val = ptr = pctx_pos(pctx);
diff --git a/libuci.c b/libuci.c
index 140edf2..786d035 100644
--- a/libuci.c
+++ b/libuci.c
@@ -148,7 +148,7 @@ uci_get_errorstr(struct uci_context *ctx, char **dest, const char *prefix)
err = UCI_ERR_UNKNOWN;
if (ctx && ctx->pctx && (err == UCI_ERR_PARSE)) {
- snprintf(error_info, sizeof(error_info) - 1, " (%s) at line %d, byte %d",
+ snprintf(error_info, sizeof(error_info) - 1, " (%s) at line %d, byte %zu",
(ctx->pctx->reason ? ctx->pctx->reason : "unknown"),
ctx->pctx->line, ctx->pctx->byte);
}
diff --git a/uci_internal.h b/uci_internal.h
index f00b394..3a94dbb 100644
--- a/uci_internal.h
+++ b/uci_internal.h
@@ -23,7 +23,7 @@ struct uci_parse_context
/* error context */
const char *reason;
int line;
- int byte;
+ size_t byte;
/* private: */
struct uci_package *package;
@@ -32,8 +32,8 @@ struct uci_parse_context
FILE *file;
const char *name;
char *buf;
- int bufsz;
- int pos;
+ size_t bufsz;
+ size_t pos;
};
#define pctx_pos(pctx) ((pctx)->pos)
#define pctx_str(pctx, i) (&(pctx)->buf[(i)])
@@ -54,7 +54,7 @@ __private struct uci_package *uci_alloc_package(struct uci_context *ctx, const c
__private FILE *uci_open_stream(struct uci_context *ctx, const char *filename, const char *origfilename, int pos, bool write, bool create);
__private void uci_close_stream(FILE *stream);
-__private void uci_getln(struct uci_context *ctx, int offset);
+__private void uci_getln(struct uci_context *ctx, size_t offset);
__private void uci_parse_error(struct uci_context *ctx, char *reason);
__private void uci_alloc_parse_context(struct uci_context *ctx);
--
2.20.1
More information about the openwrt-devel
mailing list