fix: fix network mount authentication error handling

[Kakueeen]

Fixed authentication error handling in network mount operations
by adding proper error code checking for authentication failures.
Previously, when saved login credentials failed during mount attempts,
the system would not properly detect authentication errors and would
proceed to prompt the user for credentials unnecessarily. Now correctly
identifies authentication failures (error code -8) and returns
appropriate error information.

Log: Fixed network drive mount authentication error detection

Influence:

1. Test mounting network drives with incorrect saved credentials
2. Verify authentication failure error messages are properly displayed
3. Test mount retry behavior with valid credentials after authentication
   failure
4. Check that user is only prompted for credentials when necessary

fix: 修复网络挂载认证错误处理

修复了网络挂载操作中的认证错误处理问题，增加了对认证失败错误码的正确检
查。之前当保存的登录凭据在挂载尝试中失败时，系统无法正确检测认证错误，会
不必要地提示用户输入凭据。现在能够正确识别认证失败（错误码-8）并返回适当
的错误信息。

Log: 修复网络驱动器挂载认证错误检测问题

Influence:

1. 测试使用错误的保存凭据挂载网络驱动器
2. 验证认证失败错误信息是否正确显示
3. 测试认证失败后使用有效凭据的重试挂载行为
4. 检查仅在必要时才提示用户输入凭据

BUG: https://pms.uniontech.com/bug-view-334197.html

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Kakueeen

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[deepin-ci-robot]

deepin pr auto review

这段代码主要是在处理网络挂载逻辑，当使用保存的信息挂载失败时，增加了对特定错误码（认证失败）的判断和处理。以下是对这段代码的审查意见，包括语法逻辑、代码质量、代码性能和代码安全方面：

1. 语法逻辑

• 基本正确：代码的语法逻辑是正确的，能够实现预期的功能。
• 隐式类型转换：mntRet.value(kDaemonMountRetKeyErrno).toInt() 中，如果 value 返回的 QVariant 无法转换为 int，toInt() 默认返回 0。虽然这里 0 不等于 -8，不会误判，但逻辑上不够严谨。
• 硬编码魔法数字：代码中直接使用了 -8 作为错误码，虽然注释说明了含义，但硬编码魔法数字不利于代码维护，容易出错。

2. 代码质量

• 可读性：代码结构清晰，但魔法数字 -8 降低了可读性。
• 注释：虽然有注释说明 -8 的含义，但最好使用枚举或常量来代替。
• 错误处理：只处理了 -8（认证失败）这一种错误，其他错误码未处理，可能导致错误信息不明确。

3. 代码性能

• 性能影响：这段代码对性能影响很小，主要是简单的字典查找和类型转换。
• 潜在优化：如果 mntRet 是一个频繁访问的对象，可以考虑缓存其值，但在这个场景下，挂载操作本身是 I/O 密集型，这点性能优化可以忽略。

4. 代码安全

• 类型安全：toInt() 的默认行为可能导致未预期的错误码（如 0），虽然不会影响当前逻辑，但建议增加类型检查。
• 错误处理：如果 mntRet 中没有 kDaemonMountRetKeyErrno 键，toInt() 会返回 0，可能导致错误被忽略。建议增加键存在性检查。

改进建议

1. 使用枚举或常量代替魔法数字：

   // 在头文件中定义
   enum class MountErrorCode {
       kAuthenticationFailed = -8
   };

   或

   const int kMountErrorCodeAuthFailed = -8;

2. 增加键存在性检查：

   if (mntRet.contains(kDaemonMountRetKeyErrno)) {
       int errCode = mntRet.value(kDaemonMountRetKeyErrno).toInt();
       if (errCode == static_cast<int>(MountErrorCode::kAuthenticationFailed)) {
           return { false, DeviceError::kUserErrorAuthenticationFailed, "", false };
       }
   }

3. 统一错误处理逻辑：
   如果其他错误码也需要处理，建议增加统一的错误处理函数或映射表，避免散落在各处。

改进后的代码示例

// 假设在头文件中定义了常量
const int kMountErrorCodeAuthFailed = -8;

// 在函数中
if (!mpt.isEmpty()) {
    return { true, DeviceError::kNoError, mpt };
}

if (mntRet.contains(kDaemonMountRetKeyErrno)) {
    int errCode = mntRet.value(kDaemonMountRetKeyErrno).toInt();
    if (errCode == kMountErrorCodeAuthFailed) {
        return { false, DeviceError::kUserErrorAuthenticationFailed, "", false };
    }
    // 可以在这里处理其他错误码
}

总结

这段代码在功能和逻辑上是正确的，但可以通过以下方式改进：

1. 用枚举或常量代替魔法数字。
2. 增加键存在性检查，提高代码健壮性。
3. 考虑统一错误处理逻辑，便于维护和扩展。

这些改进不会显著影响性能，但能提高代码的可读性、可维护性和健壮性。