diff options
| author | Janito Vaqueiro Ferreira Filho <janito@mullvad.net> | 2018-10-02 06:39:31 -0300 |
|---|---|---|
| committer | Janito Vaqueiro Ferreira Filho <janito@mullvad.net> | 2018-10-08 11:04:27 -0300 |
| commit | e0862e6513aede0667788e51f6125037e00f2fbf (patch) | |
| tree | a70251711d6a898e315f52daf866de90d41d9bf3 | |
| parent | 3b3fde3639de419c012bafafb51ab2e37cf04704 (diff) | |
| download | mullvadvpn-e0862e6513aede0667788e51f6125037e00f2fbf.tar.xz mullvadvpn-e0862e6513aede0667788e51f6125037e00f2fbf.zip | |
Don't persist Mssfix values that are out of range
| -rw-r--r-- | gui/packages/desktop/src/renderer/components/AdvancedSettings.js | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/gui/packages/desktop/src/renderer/components/AdvancedSettings.js b/gui/packages/desktop/src/renderer/components/AdvancedSettings.js index 9289ca7fe2..ae105d8140 100644 --- a/gui/packages/desktop/src/renderer/components/AdvancedSettings.js +++ b/gui/packages/desktop/src/renderer/components/AdvancedSettings.js @@ -168,10 +168,12 @@ export class AdvancedSettings extends Component<Props, State> { }; _onMssfixBlur = () => { - this.props.setOpenVpnMssfix(this.state.editedMssfix); - this.setState((state, _props) => { - return { focusOnMssfix: false, persistedMssfix: state.editedMssfix }; - }); + this.setState({ focusOnMssfix: false }); + + if (this._mssfixIsValid()) { + this.props.setOpenVpnMssfix(this.state.editedMssfix); + this.setState((state, _props) => ({ persistedMssfix: state.editedMssfix })); + } }; _mssfixIsValid(): boolean { |