[M94 Migration][MM] Ensure player_id is positive integer 93/291593/2
authorSuhaspoornachandra <s.poornachan@samsung.com>
Fri, 14 Oct 2022 08:27:37 +0000 (13:57 +0530)
committerBot Blink <blinkbot@samsung.com>
Wed, 19 Apr 2023 09:37:20 +0000 (09:37 +0000)
If process id is large player_id can become negative when sign bit is
set to 1 after binary operations.
This patch ensures sign bit of player id is always set to 0.

Reference:
https://review.tizen.org/gerrit/282991/

Change-Id: I4f494056587bf63b09536c6899f674dad3192705
Signed-off-by: Suhaspoornachandra <s.poornachan@samsung.com>
tizen_src/chromium_impl/content/renderer/media/tizen/renderer_media_player_manager_efl.cc

index 1816bfd85cccf8ef37fab7f3bf6d2d01d4d70649..fba339076bf5ea9d3b3ddea0ae83b9b0fed7af7f 100644 (file)
@@ -658,7 +658,7 @@ int RendererMediaPlayerManager::RegisterMediaPlayer(
   // Note : For the unique player id among the all renderer process,
   // generate player id based on renderer process id.
   static int next_media_player_id_ = base::GetCurrentProcId() << 16;
-  next_media_player_id_ = (next_media_player_id_ & 0xFFFF0000) |
+  next_media_player_id_ = (next_media_player_id_ & 0x7FFF0000) |
                           ((next_media_player_id_ + 1) & 0x0000FFFF);
   media_players_[next_media_player_id_] = player;
   return next_media_player_id_;