summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel Březina <pbrezina@redhat.com>2013-01-07 10:35:34 +0100
committerJakub Hrozek <jhrozek@redhat.com>2013-01-07 17:29:46 +0100
commitf73e8aa4b34d5be577ca7481e43b6fa87d6ed986 (patch)
tree676d6ec088b268f2855eaf883527b61748b726b9
parent09d04ff881cb9e51faff1139642793ae8c7459b3 (diff)
downloadsssd-f73e8aa4b34d5be577ca7481e43b6fa87d6ed986.tar.gz
sssd-f73e8aa4b34d5be577ca7481e43b6fa87d6ed986.tar.xz
sssd-f73e8aa4b34d5be577ca7481e43b6fa87d6ed986.zip
explicit null dereferenced in sss_nss_mc_get_record()
https://fedorahosted.org/sssd/ticket/1724
-rw-r--r--src/sss_client/nss_mc_common.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/src/sss_client/nss_mc_common.c b/src/sss_client/nss_mc_common.c
index 59f5b257b..f777890ab 100644
--- a/src/sss_client/nss_mc_common.c
+++ b/src/sss_client/nss_mc_common.c
@@ -220,6 +220,11 @@ errno_t sss_nss_mc_get_record(struct sss_cli_mc_ctx *ctx,
continue;
}
+ if (!MC_CHECK_RECORD_LENGTH(ctx, rec)) {
+ /* record has invalid length */
+ return EINVAL;
+ }
+
if (rec_len > buf_size) {
free(copy_rec);
copy_rec = malloc(rec_len);