diff --git a/src/engine/fileOps/mod.cpp b/src/engine/fileOps/mod.cpp
index 44df0a35e..8cdd4a61f 100644
--- a/src/engine/fileOps/mod.cpp
+++ b/src/engine/fileOps/mod.cpp
@@ -356,16 +356,16 @@ bool DivEngine::loadMod(unsigned char* file, size_t len) {
                 case 2: // single note slide down
                   writeFxCol(fxTyp-1+0xf1,fxVal);
                   break;
-                case 0x3: // vibrato waveform
+                case 4: // vibrato waveform
                   switch (fxVal&3) {
-                    case 0x0: // sine
+                    case 0: // sine
                       writeFxCol(0xe3,0x00);
                       break;
-                    case 0x1: // ramp down
+                    case 1: // ramp down
                       writeFxCol(0xe3,0x05);
                       break;
-                    case 0x2: // square
-                    case 0x3: 
+                    case 2: // square
+                    case 3: 
                       writeFxCol(0xe3,0x06);
                       break;
                   }
diff --git a/src/engine/fileOps/xm.cpp b/src/engine/fileOps/xm.cpp
index 3e533eb11..edc1cf14c 100644
--- a/src/engine/fileOps/xm.cpp
+++ b/src/engine/fileOps/xm.cpp
@@ -1147,7 +1147,7 @@ bool DivEngine::loadXM(unsigned char* file, size_t len) {
               case 0xe: // special...
                 // TODO: implement the rest
                 switch (effectVal>>4) {
-                  case 0x3: // vibrato waveform
+                  case 0x4: // vibrato waveform
                     switch (effectVal&3) {
                       case 0x0: // sine
                         p->data[j][effectCol[k]++]=0xe3;