I'm glad to know that Ultrix CC has a bug optimizing switch() statements

lacking an explicit `case 0:' construct. But Ultrix has been dead for more than
15 years, really. Don't give it any reason to move out of its coffin.

1 files changed, 0 insertions, 10 deletions

diff --git a/src/lib/libcrypto/rc4/rc4_enc.c b/src/lib/libcrypto/rc4/rc4_enc.c
index d8fc939dac..ec9d9e7b64 100644
--- a/src/lib/libcrypto/rc4/rc4_enc.c
+++ b/src/lib/libcrypto/rc4/rc4_enc.c
@@ -189,11 +189,6 @@ void RC4(RC4_KEY *key, size_t len, const unsigned char *indata,
                                         case 3: otp |= RC4_STEP<<i, i-=8;
                                         case 2: otp |= RC4_STEP<<i, i-=8;
                                         case 1: otp |= RC4_STEP<<i, i-=8;
-                                        case 0: ; /*
-                                                   * it's never the case,
-                                                   * but it has to be here
-                                                   * for ultrix?
-                                                   */
                                         }
                                 ochunk &= ~mask;
                                 ochunk |= (otp^ichunk) & mask;
@@ -242,11 +237,6 @@ void RC4(RC4_KEY *key, size_t len, const unsigned char *indata,
                                         case 3: otp |= RC4_STEP<<i, i+=8;
                                         case 2: otp |= RC4_STEP<<i, i+=8;
                                         case 1: otp |= RC4_STEP<<i, i+=8;
-                                        case 0: ; /*
-                                                   * it's never the case,
-                                                   * but it has to be here
-                                                   * for ultrix?
-                                                   */
                                         }
                                 ochunk &= ~mask;
                                 ochunk |= (otp^ichunk) & mask;