diff options
author | geoff <geoff> | 2000-05-31 15:28:01 +0000 |
---|---|---|
committer | geoff <geoff> | 2000-05-31 15:28:01 +0000 |
commit | 5921fd0b7920e2da872cff01b54d3b309234b902 (patch) | |
tree | 13a81fd352f79ce2d1013f9dcb0f3711cc060a09 /crypto/stack/stack.h | |
parent | a29b34593382a850e7b74145f2ee5762dbb6b387 (diff) | |
download | openssl-5921fd0b7920e2da872cff01b54d3b309234b902.tar.gz |
All the little functions created by the IMPLEMENT_STACK_OF() macro will
cast their type-specific STACK into a real STACK and call the underlying
sk_*** function. The problem is that if the STACK_OF(..) parameter being
passed in has a "const *" qualifier, it is discarded by the cast.
I'm currently implementing a fix for this but in the mean-time, this is
one case I noticed (a few type-specific sk_**_num() functions pass in
const type-specific stacks). If there are other errors in the code where
consts are being discarded, we will similarly not notice them. yuck.
Diffstat (limited to 'crypto/stack/stack.h')
-rw-r--r-- | crypto/stack/stack.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/stack/stack.h b/crypto/stack/stack.h index a6665f3b3..0cc3b44cc 100644 --- a/crypto/stack/stack.h +++ b/crypto/stack/stack.h @@ -79,7 +79,7 @@ typedef struct stack_st #define M_sk_num(sk) ((sk) ? (sk)->num:-1) #define M_sk_value(sk,n) ((sk) ? (sk)->data[n] : NULL) -int sk_num(STACK *); +int sk_num(const STACK *); char *sk_value(STACK *, int); char *sk_set(STACK *, int, char *); |