diff options
| author | Bill Parker <wp02855@gmail.com> | 2015-08-21 15:13:16 +0200 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2015-08-21 15:10:38 +0000 |
| commit | f8712aeeda620258b76efd753b5a493c8ecb5a9d (patch) | |
| tree | 2e40f1f1030e87aff2b68374d33629dc2d5d38a6 /asn1 | |
| parent | 856bab63e6c28ffc660544bfc23aca0f3d7ea341 (diff) | |
[Kerberos] Call to 'fseek()' without a check for a return value < 0,
indicating failure.
Bug: 11056
Change-Id: Iaa6b7ca07626b78c77e3c1507726ec579764a582
Reviewed-on: https://code.wireshark.org/review/10181
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'asn1')
| -rw-r--r-- | asn1/kerberos/packet-kerberos-template.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/asn1/kerberos/packet-kerberos-template.c b/asn1/kerberos/packet-kerberos-template.c index bfcbc0d605..07d7d7ec95 100644 --- a/asn1/kerberos/packet-kerberos-template.c +++ b/asn1/kerberos/packet-kerberos-template.c @@ -666,7 +666,10 @@ read_keytab_file(const char *service_key_file) sk->contents = g_memdup(buf + 2, DES3_KEY_SIZE); g_snprintf(sk->origin, KRB_MAX_ORIG_LEN, "3DES service key file, key #%d, offset %ld", count, ftell(skf)); service_key_list = g_slist_append(service_key_list, (gpointer) sk); - fseek(skf, newline_skip, SEEK_CUR); + if (fseek(skf, newline_skip, SEEK_CUR) < 0) { + fprintf(stderr, "unable to seek...\n"); + return; + } count++; } fclose(skf); |