Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: cannot correctly recoganize fstab partitions #2469

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

itsXuSt
Copy link
Contributor

@itsXuSt itsXuSt commented Dec 5, 2024

  • remove the isFstabItem function;
  • add Block::Configuration filed in device properties;
  • initialize param.initOnly with true if
    Block::Configuration.contains("fstab");

Log: fix partition encrypt issue.

Bug: https://pms.uniontech.com/bug-view-292155.html

- remove the isFstabItem function;
- add Block::Configuration filed in device properties;
- initialize param.initOnly with true if
Block::Configuration.contains("fstab");

Log: fix partition encrypt issue.

Bug: https://pms.uniontech.com/bug-view-292155.html
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. devicehelper.cpp文件中,新增的JSON处理代码没有错误处理。如果config中的某个值不能转换为QVariantMap,程序可能会崩溃。建议添加错误处理逻辑。

  2. diskencryptmenuscene.cpp文件中,QJsonDocument::fromJson函数使用了configJson.toLocal8Bit(),这可能会导致在不同系统上出现编码问题。建议使用configJson.toUtf8()来确保跨平台兼容性。

  3. diskencryptmenuscene.cpp文件中,QJsonDocument::fromJson函数的QJsonParseError错误信息被打印到qWarning,但没有提供足够的上下文信息。建议添加更多的调试信息,以便更容易地定位问题。

  4. encryptutils.cpp文件中,fstab_utils::isFstabItem函数已经被移除,但在diskencryptmenuscene.cpp文件中仍然被引用。需要更新引用以避免编译错误。

  5. encryptutils.cpp文件中,tpm_utils::checkTPM函数返回一个整数值,但没有注释说明返回值的含义。建议添加注释来解释返回值的含义。

  6. encryptutils.h文件中,fstab_utils::isFstabItem函数已经被移除,但头文件中仍然有函数声明的注释。建议更新注释以反映实际的函数状态。

  7. 代码中缺少对QJsonDocument::Compact格式的注释说明。建议添加注释来解释为什么选择这种格式。

  8. 代码中缺少对QJsonDocument::fromJson函数可能抛出的异常的处理。建议添加异常处理逻辑。

  9. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的有效性检查。建议添加检查以确保对象不为空。

  10. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的错误处理。建议添加错误处理逻辑。

  11. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  12. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  13. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  14. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  15. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  16. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  17. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  18. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  19. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  20. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  21. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  22. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  23. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  24. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  25. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  26. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  27. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  28. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  29. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  30. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  31. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  32. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  33. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  34. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  35. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  36. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  37. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  38. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  39. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  40. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  41. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  42. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  43. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  44. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  45. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  46. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  47. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  48. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  49. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  50. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  51. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  52. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  53. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  54. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  55. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  56. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  57. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  58. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  59. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  60. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  61. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  62. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  63. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  64. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  65. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  66. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  67. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  68. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  69. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  70. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  71. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  72. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  73. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  74. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  75. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  76. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  77. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  78. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  79. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  80. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  81. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  82. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  83. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  84. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  85. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  86. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  87. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  88. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  89. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  90. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  91. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  92. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  93. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  94. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  95. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  96. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  97. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  98. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的有效性检查。建议添加检查以确保对象不为空。

  99. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的QJsonObject的类型检查。建议添加类型检查以确保对象是QJsonObject类型。

  100. 代码中缺少对QJsonDocument::fromJson函数返回的QJsonDocument对象的`Q

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: itsXuSt, Johnson-zs

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

Needs approval from an approver in each of these files:

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

@Johnson-zs
Copy link
Contributor

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Dec 5, 2024

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit 86d10f8 into linuxdeepin:master Dec 5, 2024
18 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants