Skip to content
New issue

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

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

Already on GitHub? # to your account

SDL must start PTU for any app except navi right after app successfully request to start first secure service #1924

Closed
6 of 9 tasks
AStasiuk opened this issue Nov 24, 2017 · 5 comments
Labels

Comments

@AStasiuk
Copy link

AStasiuk commented Nov 24, 2017

Occurrence: Always

Steps to reproduce

  1. Make sure SDL is built with default flags
  2. Start SDL and HMI
  3. Connect mobile device
  4. Register non-navi mobile application
  5. PTU for new application without certificate is performed
  6. Start secure RPC(7) service

Actual result

SDL does not start PTU after starting secure service.

Expected result

  1. SDL starts PolicyTableUpdate process on sending SDL.OnStatusUpdate(UPDATE_NEEDED) to HMI to get "certificate" right after application successfully requests to start first secure service.
  2. SDL does not respond to StartService_request from mobile application till PTU is not finished.

Test scripts

1924_1_non-navi.lua
1924_2_navi.lua

Environment

Attachments

Expected delivery

  • Source code updates
  • Code comments
  • UTs add/update (not required)
  • ATF tests add/update
  • Manual tests (not required)
  • Add/update CI plans/jobs (not required)
  • SDD updates (not required)
  • Guidelines update (sdl_core_guides) (not required)
  • Guidelines update (sdl_hmi_integration_guidelines) (not required)
@dboltovskyi
Copy link
Contributor

Priority was set to High. Issue makes impossible starting of secure sessions because of absent certificate in PT. TLS handshake will not be resumed until valid certificate will be received.

@dboltovskyi
Copy link
Contributor

Issue is reproduced on develop branch (0b19cf4)

@jacobkeeler
Copy link
Contributor

jacobkeeler commented Jan 31, 2018

Verified that this is fixed by #1971.

@jacobkeeler
Copy link
Contributor

Test 1 fails on occasion, looks to be a timing issue

@jacobkeeler
Copy link
Contributor

Fixed with the merge of #1971

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants