test/property_test.c:339: error: BUFFER_OVERRUN_L3
  Offset: [2, +oo] Size: [1, 50] by call to `ossl_method_store_add`.
Showing all 25 steps of the trace


test/property_test.c:322:1: Array declaration
320.   }
321.   
322. > static int test_query_cache_stochastic(void)
323.   {
324.       const int max = 10000, tail = 10;

test/property_test.c:339:14: Call
337.           v[i] = 2 * i;
338.           BIO_snprintf(buf, sizeof(buf), "n=%d\n", i);
339.           if (!TEST_true(ossl_method_store_add(store, i, buf, "abc", NULL))
                    ^
340.                   || !TEST_true(ossl_method_store_cache_set(store, i, buf, v + i))
341.                   || !TEST_true(ossl_method_store_cache_set(store, i, "n=1234",

crypto/property/property.c:176:1: Parameter `*properties`
  174.   }
  175.   
  176. > int ossl_method_store_add(OSSL_METHOD_STORE *store,
  177.                             int nid, const char *properties,
  178.                             void *method, void (*method_destruct)(void *))

crypto/property/property.c:205:28: Call
  203.       ossl_method_cache_flush(store, nid);
  204.       if ((impl->properties = ossl_prop_defn_get(store->ctx, properties)) == NULL) {
  205.           impl->properties = ossl_parse_property(store->ctx, properties);
                                    ^
  206.           if (impl->properties == NULL)
  207.               goto err;

crypto/property/property_parse.c:317:1: Parameter `*defn`
    315.   }
    316.   
    317. > OSSL_PROPERTY_LIST *ossl_parse_property(OPENSSL_CTX *ctx, const char *defn)
    318.   {
    319.       PROPERTY_DEFINITION *prop = NULL;

crypto/property/property_parse.c:322:5: Assignment
    320.       OSSL_PROPERTY_LIST *res = NULL;
    321.       STACK_OF(PROPERTY_DEFINITION) *sk;
    322.       const char *s = defn;
               ^
    323.       int done;
    324.   

crypto/property/property_parse.c:328:9: Call
    326.           return NULL;
    327.   
    328.       s = skip_space(s);
                   ^
    329.       done = *s == '\0';
    330.       while (!done) {

crypto/property/property_parse.c:50:1: Parameter `*s`
      48.   DEFINE_STACK_OF(PROPERTY_DEFINITION)
      49.   
      50. > static const char *skip_space(const char *s)
      51.   {
      52.       while (ossl_isspace(*s))

crypto/property/property_parse.c:54:5: Assignment
      52.       while (ossl_isspace(*s))
      53.           s++;
      54.       return s;
                ^
      55.   }
      56.   

crypto/property/property_parse.c:328:5: Assignment
    326.           return NULL;
    327.   
    328.       s = skip_space(s);
               ^
    329.       done = *s == '\0';
    330.       while (!done) {

crypto/property/property_parse.c:335:14: Call
    333.               goto err;
    334.           memset(&prop->v, 0, sizeof(prop->v));
    335.           if (!parse_name(ctx, &s, 1, &prop->name_idx))
                        ^
    336.               goto err;
    337.           prop->oper = PROPERTY_OPER_EQ;

crypto/property/property_parse.c:81:1: Parameter `**t`
      79.   }
      80.   
      81. > static int parse_name(OPENSSL_CTX *ctx, const char *t[], int create,
      82.                         OSSL_PROPERTY_IDX *idx)
      83.   {

crypto/property/property_parse.c:342:13: Call
    340.               goto err;
    341.           }
    342.           if (match_ch(&s, '=')) {
                       ^
    343.               if (!parse_value(ctx, &s, prop, 1)) {
    344.                   PROPerr(PROP_F_OSSL_PARSE_PROPERTY, PROP_R_NO_VALUE);

crypto/property/property_parse.c:57:1: Parameter `**t`
      55.   }
      56.   
      57. > static int match_ch(const char *t[], char m)
      58.   {
      59.       const char *s = *t;

crypto/property/property_parse.c:343:18: Call
    341.           }
    342.           if (match_ch(&s, '=')) {
    343.               if (!parse_value(ctx, &s, prop, 1)) {
                            ^
    344.                   PROPerr(PROP_F_OSSL_PARSE_PROPERTY, PROP_R_NO_VALUE);
    345.                   goto err;

crypto/property/property_parse.c:245:1: Parameter `**t`
      243.   }
      244.   
      245. > static int parse_value(OPENSSL_CTX *ctx, const char *t[],
      246.                          PROPERTY_DEFINITION *res, int create)
      247.   {

crypto/property/property_parse.c:248:5: Assignment
      246.                          PROPERTY_DEFINITION *res, int create)
      247.   {
      248.       const char *s = *t;
                 ^
      249.       int r = 0;
      250.   

crypto/property/property_parse.c:252:9: Assignment
      250.   
      251.       if (*s == '"' || *s == '\'') {
      252.           s++;
                     ^
      253.           r = parse_string(ctx, &s, s[-1], res, create);
      254.       } else if (*s == '+') {

crypto/property/property_parse.c:253:13: Call
      251.       if (*s == '"' || *s == '\'') {
      252.           s++;
      253.           r = parse_string(ctx, &s, s[-1], res, create);
                         ^
      254.       } else if (*s == '+') {
      255.           s++;

crypto/property/property_parse.c:184:1: Parameter `**t`
        182.   }
        183.   
        184. > static int parse_string(OPENSSL_CTX *ctx, const char *t[], char delim,
        185.                           PROPERTY_DEFINITION *res, const int create)
        186.   {

crypto/property/property_parse.c:188:5: Assignment
        186.   {
        187.       char v[1000];
        188.       const char *s = *t;
                   ^
        189.       size_t i = 0;
        190.       int err = 0;

crypto/property/property_parse.c:205:10: Call
        203.       }
        204.       v[i] = '\0';
        205.       *t = skip_space(s + 1);
                        ^
        206.       if (err)
        207.           PROPerr(PROP_F_PARSE_STRING, PROP_R_STRING_TOO_LONG);

crypto/property/property_parse.c:50:1: <Length trace>
48.   DEFINE_STACK_OF(PROPERTY_DEFINITION)
49.   
50. > static const char *skip_space(const char *s)
51.   {
52.       while (ossl_isspace(*s))

crypto/property/property_parse.c:50:1: Parameter `*s`
          48.   DEFINE_STACK_OF(PROPERTY_DEFINITION)
          49.   
          50. > static const char *skip_space(const char *s)
          51.   {
          52.       while (ossl_isspace(*s))

crypto/property/property_parse.c:52:12: Array access: Offset: [2, +oo] Size: [1, 50] by call to `ossl_method_store_add` 
          50.   static const char *skip_space(const char *s)
          51.   {
          52.       while (ossl_isspace(*s))
                           ^
          53.           s++;
          54.       return s;

static const char *skip_space(const char *s)
{
    while (ossl_isspace(*s))
        s++;
    return s;
}
['static int test_query_cache_stochastic(void)\n{\n    const int max = 10000, tail = 10;\n    OSSL_METHOD_STORE *store;\n    int i, res = 0;\n    char buf[50];\n    void *result;\n    int errors = 0;\n    int v[10001];\n    if (!TEST_ptr(store = ossl_method_store_new(NULL))\n        || !add_property_names("n", NULL))\n        goto err;\n    for (i = 1; i <= max; i++) {\n        v[i] = 2 * i;\n        BIO_snprintf(buf, sizeof(buf), "n=%d\\n", i);\n        if (!TEST_true(ossl_method_store_add(store, i, buf, "abc", NULL))\n                || !TEST_true(ossl_method_store_cache_set(store, i, buf, v + i))\n                || !TEST_true(ossl_method_store_cache_set(store, i, "n=1234",\n                                                          "miss"))) {\n            TEST_note("iteration %d", i);\n            goto err;\n        }\n    }\n    for (i = 1; i <= max; i++) {\n        BIO_snprintf(buf, sizeof(buf), "n=%d\\n", i);\n        if (!ossl_method_store_cache_get(store, i, buf, &result)\n            || result != v + i)\n            errors++;\n    }\n    res = TEST_int_gt(errors, tail) && TEST_int_lt(errors, max - tail);\nerr:\n    ossl_method_store_free(store);\n    return res;\n}', 'int ossl_method_store_add(OSSL_METHOD_STORE *store,\n                          int nid, const char *properties,\n                          void *method, void (*method_destruct)(void *))\n{\n    ALGORITHM *alg = NULL;\n    IMPLEMENTATION *impl;\n    int ret = 0;\n    if (nid <= 0 || method == NULL || store == NULL)\n        return 0;\n    if (properties == NULL)\n        properties = "";\n    impl = OPENSSL_malloc(sizeof(*impl));\n    if (impl == NULL)\n        return 0;\n    impl->method = method;\n    impl->method_destruct = method_destruct;\n    ossl_property_write_lock(store);\n    ossl_method_cache_flush(store, nid);\n    if ((impl->properties = ossl_prop_defn_get(store->ctx, properties)) == NULL) {\n        impl->properties = ossl_parse_property(store->ctx, properties);\n        if (impl->properties == NULL)\n            goto err;\n        ossl_prop_defn_set(store->ctx, properties, impl->properties);\n    }\n    alg = ossl_method_store_retrieve(store, nid);\n    if (alg == NULL) {\n        if ((alg = OPENSSL_zalloc(sizeof(*alg))) == NULL\n                || (alg->impls = sk_IMPLEMENTATION_new_null()) == NULL\n                || (alg->cache = lh_QUERY_new(&query_hash, &query_cmp)) == NULL)\n            goto err;\n        alg->nid = nid;\n        if (!ossl_method_store_insert(store, alg))\n            goto err;\n    }\n    if (sk_IMPLEMENTATION_push(alg->impls, impl))\n        ret = 1;\n    ossl_property_unlock(store);\n    if (ret == 0)\n        impl_free(impl);\n    return ret;\nerr:\n    ossl_property_unlock(store);\n    alg_cleanup(0, alg);\n    impl_free(impl);\n    return 0;\n}', "OSSL_PROPERTY_LIST *ossl_parse_property(OPENSSL_CTX *ctx, const char *defn)\n{\n    PROPERTY_DEFINITION *prop = NULL;\n    OSSL_PROPERTY_LIST *res = NULL;\n    STACK_OF(PROPERTY_DEFINITION) *sk;\n    const char *s = defn;\n    int done;\n    if (s == NULL || (sk = sk_PROPERTY_DEFINITION_new(&pd_compare)) == NULL)\n        return NULL;\n    s = skip_space(s);\n    done = *s == '\\0';\n    while (!done) {\n        prop = OPENSSL_malloc(sizeof(*prop));\n        if (prop == NULL)\n            goto err;\n        memset(&prop->v, 0, sizeof(prop->v));\n        if (!parse_name(ctx, &s, 1, &prop->name_idx))\n            goto err;\n        prop->oper = PROPERTY_OPER_EQ;\n        if (prop->name_idx == 0) {\n            PROPerr(PROP_F_OSSL_PARSE_PROPERTY, PROP_R_PARSE_FAILED);\n            goto err;\n        }\n        if (match_ch(&s, '=')) {\n            if (!parse_value(ctx, &s, prop, 1)) {\n                PROPerr(PROP_F_OSSL_PARSE_PROPERTY, PROP_R_NO_VALUE);\n                goto err;\n            }\n        } else {\n            prop->type = PROPERTY_TYPE_STRING;\n            prop->v.str_val = ossl_property_true;\n        }\n        if (!sk_PROPERTY_DEFINITION_push(sk, prop))\n            goto err;\n        prop = NULL;\n        done = !match_ch(&s, ',');\n    }\n    if (*s != '\\0') {\n        PROPerr(PROP_F_OSSL_PARSE_PROPERTY, PROP_R_TRAILING_CHARACTERS);\n        goto err;\n    }\n    res = stack_to_property_list(sk);\nerr:\n    OPENSSL_free(prop);\n    sk_PROPERTY_DEFINITION_pop_free(sk, &pd_free);\n    return res;\n}", 'static const char *skip_space(const char *s)\n{\n    while (ossl_isspace(*s))\n        s++;\n    return s;\n}', "static int parse_name(OPENSSL_CTX *ctx, const char *t[], int create,\n                      OSSL_PROPERTY_IDX *idx)\n{\n    char name[100];\n    int err = 0;\n    size_t i = 0;\n    const char *s = *t;\n    int user_name = 0;\n    for (;;) {\n        if (!ossl_isalpha(*s)) {\n            PROPerr(PROP_F_PARSE_NAME, PROP_R_NOT_AN_IDENTIFIER);\n            return 0;\n        }\n        do {\n            if (i < sizeof(name) - 1)\n                name[i++] = ossl_tolower(*s);\n            else\n                err = 1;\n        } while (*++s == '_' || ossl_isalnum(*s));\n        if (*s != '.')\n            break;\n        user_name = 1;\n        if (i < sizeof(name) - 1)\n            name[i++] = *s;\n        else\n            err = 1;\n        s++;\n    }\n    name[i] = '\\0';\n    *t = skip_space(s);\n    if (!err) {\n        *idx = ossl_property_name(ctx, name, user_name && create);\n        return 1;\n    }\n    PROPerr(PROP_F_PARSE_NAME, PROP_R_NAME_TOO_LONG);\n    return 0;\n}", 'static int match_ch(const char *t[], char m)\n{\n    const char *s = *t;\n    if (*s == m) {\n        *t = skip_space(s + 1);\n        return 1;\n    }\n    return 0;\n}', 'static int parse_value(OPENSSL_CTX *ctx, const char *t[],\n                       PROPERTY_DEFINITION *res, int create)\n{\n    const char *s = *t;\n    int r = 0;\n    if (*s == \'"\' || *s == \'\\\'\') {\n        s++;\n        r = parse_string(ctx, &s, s[-1], res, create);\n    } else if (*s == \'+\') {\n        s++;\n        r = parse_number(&s, res);\n    } else if (*s == \'-\') {\n        s++;\n        r = parse_number(&s, res);\n        res->v.int_val = -res->v.int_val;\n    } else if (*s == \'0\' && s[1] == \'x\') {\n        s += 2;\n        r = parse_hex(&s, res);\n    } else if (*s == \'0\' && ossl_isdigit(s[1])) {\n        s++;\n        r = parse_oct(&s, res);\n    } else if (ossl_isdigit(*s)) {\n        return parse_number(t, res);\n    } else if (ossl_isalpha(*s))\n        return parse_unquoted(ctx, t, res, create);\n    if (r)\n        *t = s;\n    return r;\n}', "static int parse_string(OPENSSL_CTX *ctx, const char *t[], char delim,\n                        PROPERTY_DEFINITION *res, const int create)\n{\n    char v[1000];\n    const char *s = *t;\n    size_t i = 0;\n    int err = 0;\n    while (*s != '\\0' && *s != delim) {\n        if (i < sizeof(v) - 1)\n            v[i++] = *s;\n        else\n            err = 1;\n        s++;\n    }\n    if (*s == '\\0') {\n        PROPerr(PROP_F_PARSE_STRING,\n                PROP_R_NO_MATCHING_STRING_DELIMETER);\n        return 0;\n    }\n    v[i] = '\\0';\n    *t = skip_space(s + 1);\n    if (err)\n        PROPerr(PROP_F_PARSE_STRING, PROP_R_STRING_TOO_LONG);\n    else\n        res->v.str_val = ossl_property_value(ctx, v, create);\n    res->type = PROPERTY_TYPE_STRING;\n    return !err;\n}"]