Merge pull request #192 from deajan/gammu-fix-unlock-sms

Don't bother to check SIM security status on successful unlock
This commit is contained in:
Pierre-Lin Bonnemaison 2022-10-19 18:30:11 +02:00 committed by GitHub
commit d6b650147a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 1 deletions

View File

@ -338,7 +338,9 @@ namespace adapters;
{ {
return true; return true;
} }
// The command returns 123 on failed execution (even if SIM is already unlocked), and returns 0 if unlock was successful
// We can directly return true if command was succesful
$command_parts = [ $command_parts = [
'LC_ALL=C', 'LC_ALL=C',
'gammu', 'gammu',
@ -350,8 +352,13 @@ namespace adapters;
]; ];
$result = $this->exec_command($command_parts); $result = $this->exec_command($command_parts);
if (0 === $result['return'])
{
return true;
}
//Check security status //Check security status
// The command returns 0 regardless of the SIM security state
$command_parts = [ $command_parts = [
'LC_ALL=C', 'LC_ALL=C',
'gammu', 'gammu',