Author: mav
Date: Mon Apr 13 13:33:24 2020
New Revision: 359861
URL: https://svnweb.freebsd.org/changeset/base/359861

Log:
  MFC r359662: Relax too strict SES element descriptors check in r355430.
  
  SES specifications allows the string to be NULL-terminated, while previous
  code was considering it as invalid due to incorrectly ordered conditions.

Modified:
  stable/11/sys/cam/scsi/scsi_enc_ses.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/cam/scsi/scsi_enc_ses.c
==============================================================================
--- stable/11/sys/cam/scsi/scsi_enc_ses.c       Mon Apr 13 13:33:01 2020        
(r359860)
+++ stable/11/sys/cam/scsi/scsi_enc_ses.c       Mon Apr 13 13:33:24 2020        
(r359861)
@@ -2002,11 +2002,11 @@ ses_sanitize_elm_desc(const char *desc, uint16_t *len)
        int i;
 
        for (i = 0; i < *len; i++) {
-               if (desc[i] < 0x20 || desc[i] > 0x7e) {
+               if (desc[i] == 0) {
+                       break;
+               } else if (desc[i] < 0x20 || desc[i] > 0x7e) {
                        *len = strlen(invalid);
                        return (invalid);
-               } else if (desc[i] == 0) {
-                       break;
                }
        }
        return (desc);
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to