Project

General

Profile

« Previous | Next » 

Revision cd3cde52

Added by Jim Pingle over 6 years ago

Fix Limiter validation check, which allows old queues to display. Fixes #8956

The AQM defaults to droptail when empty, but empty was being rejected as
invalid even though it was handled in the code.

View differences:

src/etc/inc/shaper.inc
3842 3842
			$input_errors[] = gettext("Selected scheduler not recognized.");
3843 3843
		}
3844 3844
		$selectedAqm = getAQMs()[$data['aqm']];
3845
		if (!$selectedAqm) {
3845
		if (!empty($data['aqm']) && !$selectedAqm) {
3846 3846
			$input_errors[] = gettext("Selected AQM not recognized.");
3847 3847
		}
3848 3848
		if ($data['ecn'] && $data['ecn'] == 'on' && !$selectedScheduler["ecn"] && !$selectedAqm["ecn"]) {
......
4531 4531

  
4532 4532
		/* Limiter patch */
4533 4533
		$selectedAqm = getAQMs()[$data['aqm']];
4534
		if (!$selectedAqm) {
4534
		if (!empty($data['aqm']) && !$selectedAqm) {
4535 4535
			$input_errors[] = gettext("Selected AQM not recognized.");
4536 4536
		}
4537 4537
		if ($data['ecn'] && $data['ecn'] == 'on' && !$selectedAqm["ecn"]) {

Also available in: Unified diff