From a5341beaa50299148c3498a98f051625963c350c Mon Sep 17 00:00:00 2001
From: Xulunix <git@xulunix.de>
Date: Thu, 3 Jun 2021 02:26:24 +0200
Subject: [PATCH]  Fixed bug in onvifprobe

The arguments for "/usr/bin/zmonvif-probe.pl" was always `probe 1.1,1.2 1` with the last argument always being `1` instead of the network interface name selected in the menu.
This change fixes this issue by using the selected interface name instead of the boolean return value of the `isset()` function.
---
 web/skins/classic/views/onvifprobe.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/skins/classic/views/onvifprobe.php b/web/skins/classic/views/onvifprobe.php
index 70ad8ead9..754bb50ef 100644
--- a/web/skins/classic/views/onvifprobe.php
+++ b/web/skins/classic/views/onvifprobe.php
@@ -50,7 +50,7 @@ function execONVIF($cmd) {
 
 function probeCameras($localIp) {
   $cameras = array();
-  $lines = @execONVIF('probe 1.1,1.2'.(isset($_REQUEST['interface']) ? ' '.isset($_REQUEST['interface']) : '' ));
+  $lines = @execONVIF('probe 1.1,1.2'.(isset($_REQUEST['interface']) ? ' '.$_REQUEST['interface'] : '' ));
   if ( $lines ) {
     foreach ( $lines as $line ) {
       $line = rtrim($line);