Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 9 Dec 2022 14:13:22 GMT
From:      Cy Schubert <cy@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 4ed9d88da492 - stable/12 - Revert "heimdal: Fix bus fault when zero-length request received"
Message-ID:  <202212091413.2B9EDMDa058359@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by cy:

URL: https://cgit.FreeBSD.org/src/commit/?id=4ed9d88da4923be646a98f471323ee3b7a080e4e

commit 4ed9d88da4923be646a98f471323ee3b7a080e4e
Author:     Cy Schubert <cy@FreeBSD.org>
AuthorDate: 2022-12-08 16:08:50 +0000
Commit:     Cy Schubert <cy@FreeBSD.org>
CommitDate: 2022-12-09 14:13:09 +0000

    Revert "heimdal: Fix bus fault when zero-length request received"
    
    This is the wrong fix for PR/268062.
    
    This reverts commit 6742ff42ab3b6e65239f975314060b1393e22d62.
    
    (cherry picked from commit 5319949668a5cb4d182d7252ce26799506b3585d)
---
 crypto/heimdal/lib/krb5/read_message.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/crypto/heimdal/lib/krb5/read_message.c b/crypto/heimdal/lib/krb5/read_message.c
index e994b0f09133..4e9bd012dd67 100644
--- a/crypto/heimdal/lib/krb5/read_message.c
+++ b/crypto/heimdal/lib/krb5/read_message.c
@@ -55,11 +55,6 @@ krb5_read_message (krb5_context context,
 	return HEIM_ERR_EOF;
     }
     len = (buf[0] << 24) | (buf[1] << 16) | (buf[2] << 8) | buf[3];
-    if (len == 0) {
-	krb5_clear_error_message(context);
-	return HEIM_ERR_EOF;
-    }
-
     ret = krb5_data_alloc (data, len);
     if (ret) {
 	krb5_clear_error_message(context);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202212091413.2B9EDMDa058359>