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.

Sign up for GitHub

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

Jump to bottom

pick some recently fixes #2524

Draft
wants to merge 8 commits into
base: release/v23
Choose a base branch
from
Draft

pick some recently fixes #2524

wants to merge 8 commits into from

Conversation

itsXuSt
Copy link
Contributor

@itsXuSt itsXuSt commented Dec 26, 2024

  • fix: [mount] passwd not clear when all unmounted.
  • refactor: improve network mount password handling
  • fix: refactoring the deletion logic for temporary passwords
  • fix: add dmask fmask when mount device
  • fix: Remove view mode settings when reset view mode
  • fix: save last visited url in filedialog.

max-lvs
max-lvs previously approved these changes Dec 26, 2024
View reviewed changes
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见：

  1. 在application.cpp文件中，appAttributeTrigger函数中，continue语句后缺少分号，应添加分号以避免语法错误。
  2. 在devicemanager.cpp文件中，mountBlockDevAsync函数中，options变量在函数内部被修改，但未在函数外部使用，建议移除options变量的定义。
  3. 在devicemanager.cpp文件中，askForPasswdWhenMountNetworkDevice函数中，info.cancelled = true;后缺少return语句，应添加以避免逻辑错误。
  4. 在mountaskpassworddialog.cpp文件中，handleConnect函数中，loginObj.insert(kPasswdSaveMode, kSaveBeforeLogout);应改为loginObj.insert(kPasswdSaveMode, kNeverSave);以符合注释中的描述。
  5. 在filedialoghandle.cpp文件中，FileDialogHandle构造函数中，defaultUrl变量未在else分支中初始化，可能导致未定义行为。
  6. 在filedialog.cpp文件中，FileDialogPrivate构造函数中，lastVisitedDir变量未在else分支中初始化，可能导致未定义行为。
  7. 在filedialog.cpp文件中，FileDialog::cd函数中，d->lastVisitedDir = url;应在if语句块中，以避免在else分支中重复设置。
  8. 在filedialog.cpp文件中，FileDialog::initConnect函数中，connect语句中缺少using namespace，应添加以避免命名空间冲突。
  9. 在fileview.cpp文件中，FileView::onDefaultViewModeChanged函数中，if (newMode == d->currentViewMode)应改为if (newMode == d->currentViewMode)以避免逻辑错误。
  10. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_new_full函数的参数顺序错误，应将g_str_hash和g_str_equal参数位置互换。
  11. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  12. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  13. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  14. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  15. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  16. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  17. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  18. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  19. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  20. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  21. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  22. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  23. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  24. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  25. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  26. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  27. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  28. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  29. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  30. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  31. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  32. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  33. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  34. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  35. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  36. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  37. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  38. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  39. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  40. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  41. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  42. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  43. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  44. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  45. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  46. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  47. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  48. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  49. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  50. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  51. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  52. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  53. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  54. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  55. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  56. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  57. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  58. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  59. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  60. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  61. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  62. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  63. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  64. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  65. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  66. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  67. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  68. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  69. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  70. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  71. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  72. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  73. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  74. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  75. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  76. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  77. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  78. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  79. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  80. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  81. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  82. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession函数中，g_hash_table_insert函数的参数顺序错误，应将g_strdup函数的参数顺序互换。
  83. 在protocoldisplayutilities.cpp文件中，secret_utils::forgetPasswordInSession

@itsXuSt itsXuSt marked this pull request as draft December 26, 2024 12:10
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: itsXuSt

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

1 similar comment
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: itsXuSt

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

@itsXuSt
fix: save last visited url in filedialog.
f9b50e6 
- if directory is not setted when using FileDialog, use lastVisitedUrl
as default;

Log: as above.
@itsXuSt
fix: [mount] passwd not clear when all unmounted.
7d89511 
saved password should be cleared when all associated smb share
unmounted.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-275557.html
Change-Id: Ia2434bd4f2c47594aba24b6dbfa554cb324116a2
@itsXuSt
refactor: improve network mount password handling
3f5c454 
- Refactor password saving logic for network mounts
- Change default password save mode to 'Never Save' instead of 'Save
Before Logout'
- Add temporary configuration file to track SMB password save mode
- Optimize password cleanup when unmounting SMB shares

This change improves security by making password saving more explicit
and
cleaning up credentials properly when unmounting network shares.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-279371.html
Change-Id: I09833084aba2f242d0e63502830dbe72e34846e7
itsXuSt added 5 commits January 3, 2025 16:21
@itsXuSt
fix: refactoring the deletion logic for temporary passwords
5125e51 
- clear the password when the last smb of same host unmounted;
- only clear the password saved in `session` collection.

Log: fix samba password clean issue

Bug: https://pms.uniontech.com/bug-view-294555.html
Change-Id: I675584cff7907956d10b70889c0ecb71f5aaabbf
@itsXuSt
fix: size not updated in admin mode.
0c9dd38 
size update timer is not running in admin mode.

Log: as above.

Bug: https://pms.uniontech.com/bug-view-278679.html
Change-Id: I5ae46ce6362656904e2ff5f74afa94d7bcdd45b7
@itsXuSt
fix: move burn confirm dialog center.
7378168 
show dialogs in cursor screen's center.

Log: as above.
@itsXuSt
fix: add dmask fmask when mount device
23d44e4 
only for vfat/ntfs/exfat fs. make the mountpoint share in different
users

Log: as above.

Change-Id: Iffc0315f65edc3d605b9cc730f799e181c0d2494
@itsXuSt
fix: Remove view mode settings when reset view mode
001d5e7 
When resetting view mode, remove the view mode settings from
FileViewState instead of setting it to default value. This ensures the
view mode settings are completely cleared.

Log: Fix view mode reset behavior by removing settings instead of
setting default value

Bug: https://pms.uniontech.com/bug-view-286389.html
Change-Id: Ib5a6c04bd4ce1a8d6a81b985f4574f02b50c5f98
@itsXuSt itsXuSt force-pushed the v23 branch 2 times, most recently from 913c80d to 001d5e7 Compare January 3, 2025 09:01
@deepin-bot
Copy link
Contributor

deepin-bot bot commented Feb 7, 2025

TAG Bot

New tag: 6.5.10.3
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #2563

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@max-lvs max-lvs max-lvs left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@itsXuSt @deepin-ci-robot @max-lvs