Add validation checks for channel count and file size in BinaryFileSource

Author: anjaldoshi

Plugins/LfpViewer/LfpDisplayOptions.cpp

@@ -1268,6 +1268,11 @@ void LfpDisplayOptions::comboBoxChanged (ComboBox* cb)
         selectedVoltageRange[selectedChannelType] = cb->getSelectedId();
         selectedVoltageRangeValues[selectedChannelType] = cb->getText();
         canvasSplit->redraw();
+
+        if (selectedChannelType == ContinuousChannel::Type::AUX && isAuxAutoScaleEnabled())
+            rangeSelectionLabel->setText ("Range", dontSendNotification);
+        else
+            rangeSelectionLabel->setText ("Range (" + rangeUnits[selectedChannelType] + ")", dontSendNotification);
     }
     else if (cb == spreadSelection.get())
     {

Source/Processors/FileReader/BinaryFileSource/BinaryFileSource.cpp

@@ -131,6 +131,21 @@ void BinaryFileSource::fillRecordInfo()
         }
 
         int numChannels = record[idNumChannels];
+
+        // if numchannels is not equal to the size of channels var, skip this record
+        if (numChannels != channels.size())
+        {
+            LOGE ("Number of channels mismatch in stream: ", streamName);
+            continue;
+        }
+
+        // if numSamples is not a whole number, skip this record
+        if (dataFile.getSize() % (numChannels * sizeof (int16)) != 0)
+        {
+            LOGE ("File size is not consistent with number of channels in stream: ", streamName);
+            continue;
+        }
+
         int64 numSamples = (dataFile.getSize() / numChannels) / sizeof (int16);
 
         info.name = streamName;

Source/Processors/FileReader/FileReader.cpp

@@ -286,7 +286,7 @@ bool FileReader::setFile (String fullpath, bool shouldUpdateSignalChain)
     if (isEmptyFile)
     {
         input = nullptr;
-        showWarningAsync ("Failed to open file", "Continuous data file is missing or empty.");
+        showWarningAsync ("Failed to open file", "Continuous data file is missing, empty, or invalid.");
         CoreServices::sendStatusMessage ("Empty file. Ignoring open operation");
 
         return false;