improve str_to_buf and buf_to_str helpers
[openocd.git] / src / helper / binarybuffer.c
index f2ca4719c39fb693847849051d118dcbd6654499..0def9485530afd834cf956b71b4efd4926bd05b4 100644 (file)
@@ -2,7 +2,7 @@
  *   Copyright (C) 2004, 2005 by Dominic Rath                              *
  *   Dominic.Rath@gmx.de                                                   *
  *                                                                         *
- *   Copyright (C) 2007,2008 Øyvind Harboe                                 *
+ *   Copyright (C) 2007,2008 Øyvind Harboe                                 *
  *   oyvind.harboe@zylin.com                                               *
  *                                                                         *
  *   This program is free software; you can redistribute it and/or modify  *
@@ -50,13 +50,10 @@ const unsigned char bit_reverse_table256[] =
 
 uint8_t* buf_cpy(const uint8_t *from, uint8_t *to, int size)
 {
-       unsigned int num_bytes = CEIL(size, 8);
-       unsigned int i;
-
        if (from == NULL)
                return NULL;
 
-       for (i = 0; i < num_bytes; i++)
+       for (unsigned i = 0, num_bytes = CEIL(size, 8); i < num_bytes; i++)
                to[i] = from[i];
 
        /* mask out bits that don't belong to the buffer */
@@ -70,17 +67,14 @@ uint8_t* buf_cpy(const uint8_t *from, uint8_t *to, int size)
 
 int buf_cmp(const uint8_t *buf1, const uint8_t *buf2, int size)
 {
-       int num_bytes = CEIL(size, 8);
-       int i;
-
        if (!buf1 || !buf2)
                return 1;
 
-       for (i = 0; i < num_bytes; i++)
+       for (unsigned i = 0, num_bytes = CEIL(size, 8); i < num_bytes; i++)
        {
                /* last byte */
                /* mask out bits that don't really belong to the buffer if size isn't a multiple of 8 bits */
-               if ((size % 8) && (i == num_bytes -1 ))
+               if ((size % 8) && (i == num_bytes -1))
                {
                        if ((buf1[i] & ((1 << (size % 8)) - 1)) != (buf2[i] & ((1 << (size % 8)) - 1)))
                                return 1;
@@ -97,14 +91,11 @@ int buf_cmp(const uint8_t *buf1, const uint8_t *buf2, int size)
 
 int buf_cmp_mask(const uint8_t *buf1, const uint8_t *buf2, const uint8_t *mask, int size)
 {
-       int num_bytes = CEIL(size, 8);
-       int i;
-
-       for (i = 0; i < num_bytes; i++)
+       for (unsigned i = 0, num_bytes = CEIL(size, 8); i < num_bytes; i++)
        {
                /* last byte */
                /* mask out bits that don't really belong to the buffer if size isn't a multiple of 8 bits */
-               if ((size % 8) && (i == num_bytes -1 ))
+               if ((size % 8) && (i == num_bytes -1))
                {
                        if ((buf1[i] & ((1 << (size % 8)) - 1) & mask[i]) !=
                                (buf2[i] & ((1 << (size % 8)) - 1) & mask[i]))
@@ -122,10 +113,7 @@ int buf_cmp_mask(const uint8_t *buf1, const uint8_t *buf2, const uint8_t *mask,
 
 uint8_t* buf_set_ones(uint8_t *buf, int count)
 {
-       int num_bytes = CEIL(count, 8);
-       int i;
-
-       for (i = 0; i < num_bytes; i++)
+       for (unsigned i = 0, num_bytes = CEIL(count, 8); i < num_bytes; i++)
        {
                if (count >= 8)
                        buf[i] = 0xff;
@@ -141,14 +129,13 @@ uint8_t* buf_set_ones(uint8_t *buf, int count)
 uint8_t* buf_set_buf(const uint8_t *src, int src_start, uint8_t *dst, int dst_start, int len)
 {
        int src_idx = src_start, dst_idx = dst_start;
-       int i;
 
-       for (i = 0; i < len; i++)
+       for (int i = 0; i < len; i++)
        {
-               if (((src[src_idx/8] >> (src_idx % 8)) & 1) == 1)
-                       dst[dst_idx/8] |= 1 << (dst_idx%8);
+               if (((src[src_idx / 8] >> (src_idx % 8)) & 1) == 1)
+                       dst[dst_idx / 8] |= 1 << (dst_idx % 8);
                else
-                       dst[dst_idx/8] &= ~(1 << (dst_idx%8));
+                       dst[dst_idx / 8] &= ~(1 << (dst_idx % 8));
                dst_idx++;
                src_idx++;
        }
@@ -156,11 +143,9 @@ uint8_t* buf_set_buf(const uint8_t *src, int src_start, uint8_t *dst, int dst_st
        return dst;
 }
 
-u32 flip_u32(u32 value, unsigned int num)
+uint32_t flip_u32(uint32_t value, unsigned int num)
 {
-       u32 c;
-
-       c = (bit_reverse_table256[value & 0xff] << 24) |
+       uint32_t c = (bit_reverse_table256[value & 0xff] << 24) |
                (bit_reverse_table256[(value >> 8) & 0xff] << 16) |
                (bit_reverse_table256[(value >> 16) & 0xff] << 8) |
                (bit_reverse_table256[(value >> 24) & 0xff]);
@@ -173,12 +158,10 @@ u32 flip_u32(u32 value, unsigned int num)
 
 int ceil_f_to_u32(float x)
 {
-       u32 y;
-
        if (x < 0)      /* return zero for negative numbers */
                return 0;
 
-       y = x;  /* cut off fraction */
+       uint32_t y = x; /* cut off fraction */
 
        if ((x - y) > 0.0) /* if there was a fractional part, increase by one */
                y++;
@@ -186,88 +169,86 @@ int ceil_f_to_u32(float x)
        return y;
 }
 
-char* buf_to_str(const uint8_t *buf, int buf_len, int radix)
+char* buf_to_str(const void *_buf, unsigned buf_len, unsigned radix)
 {
-       const char *DIGITS = "0123456789ABCDEF";
        float factor;
-       char *str;
-       int str_len;
-       int b256_len = CEIL(buf_len, 8);
-       u32 tmp;
-
-       int j; /* base-256 digits */
-       int i; /* output digits (radix) */
-
-       if (radix == 16)
-       {
+       switch (radix) {
+       case 16:
                factor = 2.0;   /* log(256) / log(16) = 2.0 */
-       }
-       else if (radix == 10)
-       {
+               break;
+       case 10:
                factor = 2.40824;   /* log(256) / log(10) = 2.40824 */
-       }
-       else if (radix == 8)
-       {
+               break;
+       case 8:
                factor = 2.66667;       /* log(256) / log(8) = 2.66667 */
-       }
-       else
+               break;
+       default:
                return NULL;
+       }
 
-       str_len = ceil_f_to_u32(CEIL(buf_len, 8) * factor);
-       str = calloc(str_len + 1, 1);
+       unsigned str_len = ceil_f_to_u32(CEIL(buf_len, 8) * factor);
+       char *str = calloc(str_len + 1, 1);
 
-       for (i = b256_len - 1; i >= 0; i--)
+       const uint8_t *buf = _buf;
+       int b256_len = CEIL(buf_len, 8);
+       for (int i = b256_len - 1; i >= 0; i--)
        {
-               tmp = buf[i];
-               if ((i == (buf_len / 8)) && (buf_len % 8))
+               uint32_t tmp = buf[i];
+               if (((unsigned)i == (buf_len / 8)) && (buf_len % 8))
                        tmp &= (0xff >> (8 - (buf_len % 8)));
 
-               for (j = str_len; j > 0; j--)
+               /* base-256 digits */
+               for (unsigned j = str_len; j > 0; j--)
                {
-                       tmp += (u32)str[j-1] * 256;
+                       tmp += (uint32_t)str[j-1] * 256;
                        str[j-1] = (uint8_t)(tmp % radix);
                        tmp /= radix;
                }
        }
 
-       for (j = 0; j < str_len; j++)
+       const char *DIGITS = "0123456789ABCDEF";
+       for (unsigned j = 0; j < str_len; j++)
                str[j] = DIGITS[(int)str[j]];
 
        return str;
 }
 
-int str_to_buf(const char *str, int str_len, uint8_t *buf, int buf_len, int radix)
+/// identify radix, and skip radix-prefix (0, 0x or 0X)
+static void str_radix_guess(const char **_str, unsigned *_str_len,
+               unsigned *_radix)
 {
-       char *charbuf;
-       u32 tmp;
-       float factor;
-       uint8_t *b256_buf;
-       int b256_len;
-
-       int j; /* base-256 digits */
-       int i; /* input digits (ASCII) */
-
-       if (radix == 0)
+       unsigned radix = *_radix;
+       if (0 != radix)
+               return;
+       const char *str = *_str;
+       unsigned str_len = *_str_len;   
+       if (str[0] == '0' && (str[1] == 'x' || str[1] == 'X'))
        {
-               /* identify radix, and skip radix-prefix (0, 0x or 0X) */
-               if ((str[0] == '0') && (str[1] && ((str[1] == 'x') || (str[1] == 'X'))))
-               {
-                       radix = 16;
-                       str += 2;
-                       str_len -= 2;
-               }
-               else if ((str[0] == '0') && (str_len != 1))
-               {
-                       radix = 8;
-                       str += 1;
-                       str_len -= 1;
-               }
-               else
-               {
-                       radix = 10;
-               }
+               radix = 16;
+               str += 2;
+               str_len -= 2;
        }
+       else if ((str[0] == '0') && (str_len != 1))
+       {
+               radix = 8;
+               str += 1;
+               str_len -= 1;
+       }
+       else
+       {
+               radix = 10;
+       }
+       *_str = str;
+       *_str_len = str_len;
+       *_radix = radix;
+}
 
+int str_to_buf(const char *str, unsigned str_len,
+               void *_buf, unsigned buf_len, unsigned radix)
+{
+       str_radix_guess(&str, &str_len, &radix);
+
+       float factor;
        if (radix == 16)
                factor = 0.5; /* log(16) / log(256) = 0.5 */
        else if (radix == 10)
@@ -278,18 +259,20 @@ int str_to_buf(const char *str, int str_len, uint8_t *buf, int buf_len, int radi
                return 0;
 
        /* copy to zero-terminated buffer */
-       charbuf = malloc(str_len + 1);
+       char *charbuf = malloc(str_len + 1);
        memcpy(charbuf, str, str_len);
        charbuf[str_len] = '\0';
 
        /* number of digits in base-256 notation */
-       b256_len = ceil_f_to_u32(str_len * factor);
-       b256_buf = calloc(b256_len, 1);
+       unsigned b256_len = ceil_f_to_u32(str_len * factor);
+       uint8_t *b256_buf = calloc(b256_len, 1);
 
        /* go through zero terminated buffer */
+       /* input digits (ASCII) */
+       unsigned i;
        for (i = 0; charbuf[i]; i++)
        {
-               tmp = charbuf[i];
+               uint32_t tmp = charbuf[i];
                if ((tmp >= '0') && (tmp <= '9'))
                        tmp = (tmp - '0');
                else if ((tmp >= 'a') && (tmp <= 'f'))
@@ -298,19 +281,21 @@ int str_to_buf(const char *str, int str_len, uint8_t *buf, int buf_len, int radi
                        tmp = (tmp - 'A' + 10);
                else continue;  /* skip characters other than [0-9,a-f,A-F] */
 
-               if (tmp >= (u32)radix)
+               if (tmp >= radix)
                        continue;       /* skip digits invalid for the current radix */
 
-               for (j = 0; j < b256_len; j++)
+               /* base-256 digits */
+               for (unsigned j = 0; j < b256_len; j++)
                {
-                       tmp += (u32)b256_buf[j] * radix;
+                       tmp += (uint32_t)b256_buf[j] * radix;
                        b256_buf[j] = (uint8_t)(tmp & 0xFF);
                        tmp >>= 8;
                }
 
        }
 
-       for (j = 0; j < CEIL(buf_len, 8); j++)
+       uint8_t *buf = _buf;
+       for (unsigned j = 0; j < CEIL(buf_len, 8); j++)
        {
                if (j < b256_len)
                        buf[j] = b256_buf[j];
@@ -327,12 +312,3 @@ int str_to_buf(const char *str, int str_len, uint8_t *buf, int buf_len, int radi
 
        return i;
 }
-
-int buf_to_u32_handler(uint8_t *in_buf, void *priv, struct scan_field_s *field)
-{
-       u32 *dest = priv;
-
-       *dest = buf_get_u32(in_buf, 0, 32);
-
-       return ERROR_OK;
-}

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)