[PR #121] [MERGED] Tabbar::GetTargetTab() fix: bounds-checking implemented (crash fix). #168

Closed
opened 2026-05-05 03:42:03 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ultimatepp/ultimatepp/pull/121
Author: @ismail-yilmaz
Created: 12/24/2022
Status: Merged
Merged: 12/24/2022
Merged by: @mirek-fidler

Base: masterHead: tabbarfix


📝 Commits (1)

  • ce210ea Tabbar::GetTabBar(): bounds-checking implemented.

📊 Changes

1 file changed (+6 additions, -2 deletions)

View changed files

📝 uppsrc/TabBar/TabBar.cpp (+6 -2)

📄 Description

Fixes a crash due to out-of-bounds access (access violation), when dragging stuff over an empty tabbar.
Original report: https://www.ultimatepp.org/forums/index.php?t=msg&th=11990&goto=59424&#msg_59424


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ultimatepp/ultimatepp/pull/121 **Author:** [@ismail-yilmaz](https://github.com/ismail-yilmaz) **Created:** 12/24/2022 **Status:** ✅ Merged **Merged:** 12/24/2022 **Merged by:** [@mirek-fidler](https://github.com/mirek-fidler) **Base:** `master` ← **Head:** `tabbarfix` --- ### 📝 Commits (1) - [`ce210ea`](https://github.com/ultimatepp/ultimatepp/commit/ce210ea922448e827108f1d5743c12e702e92dbc) Tabbar::GetTabBar(): bounds-checking implemented. ### 📊 Changes **1 file changed** (+6 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `uppsrc/TabBar/TabBar.cpp` (+6 -2) </details> ### 📄 Description Fixes a crash due to out-of-bounds access (access violation), when dragging stuff over an empty tabbar. Original report: https://www.ultimatepp.org/forums/index.php?t=msg&th=11990&goto=59424&#msg_59424 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 03:42:03 -06:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/ultimatepp#168
No description provided.