Commit edb2c34f authored by David S. Miller's avatar David S. Miller

[NETFILTER]: Fix warnings in ip_nat_snmp_basic.c

net/ipv4/netfilter/ip_nat_snmp_basic.c: In function 'asn1_header_decode':
net/ipv4/netfilter/ip_nat_snmp_basic.c:248: warning: 'len' may be used uninitialized in this function
net/ipv4/netfilter/ip_nat_snmp_basic.c:248: warning: 'def' may be used uninitialized in this function
net/ipv4/netfilter/ip_nat_snmp_basic.c: In function 'snmp_translate':
net/ipv4/netfilter/ip_nat_snmp_basic.c:672: warning: 'l' may be used uninitialized in this function
net/ipv4/netfilter/ip_nat_snmp_basic.c:668: warning: 'type' may be used uninitialized in this function
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent fb950496
...@@ -250,6 +250,7 @@ static unsigned char asn1_header_decode(struct asn1_ctx *ctx, ...@@ -250,6 +250,7 @@ static unsigned char asn1_header_decode(struct asn1_ctx *ctx,
if (!asn1_id_decode(ctx, cls, con, tag)) if (!asn1_id_decode(ctx, cls, con, tag))
return 0; return 0;
def = len = 0;
if (!asn1_length_decode(ctx, &def, &len)) if (!asn1_length_decode(ctx, &def, &len))
return 0; return 0;
...@@ -669,7 +670,7 @@ static unsigned char snmp_object_decode(struct asn1_ctx *ctx, ...@@ -669,7 +670,7 @@ static unsigned char snmp_object_decode(struct asn1_ctx *ctx,
unsigned char *eoc, *end, *p; unsigned char *eoc, *end, *p;
unsigned long *lp, *id; unsigned long *lp, *id;
unsigned long ul; unsigned long ul;
long l; long l;
*obj = NULL; *obj = NULL;
id = NULL; id = NULL;
...@@ -699,11 +700,13 @@ static unsigned char snmp_object_decode(struct asn1_ctx *ctx, ...@@ -699,11 +700,13 @@ static unsigned char snmp_object_decode(struct asn1_ctx *ctx,
return 0; return 0;
} }
type = 0;
if (!snmp_tag_cls2syntax(tag, cls, &type)) { if (!snmp_tag_cls2syntax(tag, cls, &type)) {
kfree(id); kfree(id);
return 0; return 0;
} }
l = 0;
switch (type) { switch (type) {
case SNMP_INTEGER: case SNMP_INTEGER:
len = sizeof(long); len = sizeof(long);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment