Skip to content

I am receiving a report regarding a security vulnerability. #651

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

Closed
datpt11 opened this issue Mar 12, 2025 · 5 comments · May be fixed by #666
Closed

I am receiving a report regarding a security vulnerability. #651

datpt11 opened this issue Mar 12, 2025 · 5 comments · May be fixed by #666

Comments

@datpt11
Copy link

datpt11 commented Mar 12, 2025

com.hiennv.flutter_callkit_incoming.CallkitIncomingActivity Risk: Missing protection against task hijacking. <activity android:name="com.hiennv.flutter_callkit_incoming.CallkitIncomingActivity" android:exported="true" android:taskAffinity="com.hiennv.flutter_callkit_incoming.INCOMING_CALL_AFFINITY" />

@thaidmfinnick
Copy link

thaidmfinnick commented Mar 19, 2025

@hiennguyen92 với bản 2.5.1 em test Android nếu gọi khi tắt màn hình thì k hiện màn hình callkit incoming ạ, mở màn hình thì vẫn hiện bình thường ạ.

bản 2.5.0 thì vẫn hiện trong cả 2 trường hợp (tắt và mở màn hình) ạ

@hiennguyen92
Copy link
Owner

hiennguyen92 commented Mar 19, 2025

lạ nhỉ. chỉ là không exported nó ra nữa thôi mà nhỉ.

@thaidmfinnick
Copy link

thaidmfinnick commented Mar 20, 2025

Em cũng thấy lạ, em có test thêm với điện thoại khác (cả máy thật và máy ảo) thì kết quả như này ạ:

Ở bản 2.5.1:

  • android 11 (realme 5 pro): vẫn có thể show callkit ở cả lúc khoá và lúc mở màn hình
  • android 14 (samsung galaxy A05): khoá màn hình là không hiện full screen callkit, lúc mở màn hình noti callkit đến, ấn vào noti cũng không hiện full screen
  • android 14 (máy ảo): tương tự như samsung galaxy A05

Ở bản 2.5.0:
Cả android 11 và 14, lúc khoá và mở đều hiện noti callkit incoming

@td2thinh
Copy link

I have made a PR regarding this, please check it out: #666

P/s: sry no Unikey on work pc.

@thaidmfinnick
Copy link

Thanks bro, let me test and give you a feedback

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 a pull request may close this issue.

4 participants