Re: [PATCH] extrahd.cgi: Fix broken UUID check

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1402 bytes --]

Thank you for working on this so urgently.

> On 12 Sep 2023, at 19:20, Stefan Schantl <stefan.schantl(a)ipfire.org> wrote:
> 
> This check was totaly broken and resulted into not beeing able to
> configure/mount more than one extra harddrive.
> 
> Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
> ---
> html/cgi-bin/extrahd.cgi | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/html/cgi-bin/extrahd.cgi b/html/cgi-bin/extrahd.cgi
> index 9cdfc69f8..c3a14c9e4 100644
> --- a/html/cgi-bin/extrahd.cgi
> +++ b/html/cgi-bin/extrahd.cgi
> @@ -113,13 +113,16 @@ if ($extrahdsettings{'ACTION'} eq $Lang::tr{'add'}) {
> # Split the line into pieces and assign nice variables.
> my ($uuid, $fs, $path) = split( /\;/, $entry );
> 
> + # Remove tailing UUID= from uuid string.
> + $uuid =~ s{^UUID=}{};
> +
> # Check if the path is allready used.
> if ( "$extrahdsettings{'PATH'}" eq "$path" ) {
> $errormessage = "$Lang::tr{'extrahd you cant mount'} $extrahdsettings{'DEVICE'} $Lang::tr{'extrahd to'} $extrahdsettings{'PATH'} $Lang::tr{'extrahd because there is already a device mounted'}.";
> }
> 
> # Check if the uuid is allready used.
> - if ("$extrahdsettings{'DEVICE'} eq $uuid") {
> + if ("$extrahdsettings{'UUID'}" eq "$uuid") {
> $errormessage = "$extrahdsettings{'DEVICE'} is allready mounted.";
> }
> }
> -- 
> 2.39.2
>