FIX: S360 Resolve [CodeQL.SM02986] 'Cast from char* to wchar_t*

[gargsaumya]

Work Item / Issue Reference

AB#40019
AB#40020

GitHub Issue: #<ISSUE_NUMBER>

---

Summary

This pull request adds explanatory comments to several reinterpret_cast statements in the FetchLobColumnData function within mssql_python/pybind/ddbc_bindings.cpp, clarifying the safety of these casts for CodeQL static analysis. The comments explain why casting from std::vector data to wide character pointers is safe in this context.

Code safety and documentation improvements:

• Added detailed comments to reinterpret_cast<const SQLWCHAR*>(chunk.data()) and similar casts, explaining alignment guarantees and safe usage for CodeQL [SM02986] in both Windows and Linux/macOS code paths. [1] [2]

[Copilot]

Pull Request Overview

This PR adds suppression comments for CodeQL static analysis warnings (SM02986) related to reinterpret_cast operations when converting raw byte buffers to wide character types (SQLWCHAR* and wchar_t*) in the LOB data fetching function.

Key changes:

• Added inline comments justifying the safety of three reinterpret_cast operations
• Comments explain alignment guarantees from std::vector and ODBC driver behavior

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

📊 Code Coverage Report

🔥 Diff Coverage 100%

🎯 Overall Coverage 77%

📈 Total Lines Covered: 4617 out of 5987 📁 Project: mssql-python

---

Diff Coverage

Diff: main...HEAD, staged and unstaged changes

• mssql_python/pybind/ddbc_bindings.cpp (100%)

Summary

• Total: 8 lines
• Missing: 0 lines
• Coverage: 100%

---

📋 Files Needing Attention

📉 Files with overall lowest coverage (click to expand)

mssql_python.pybind.ddbc_bindings.cpp: 70.9%
mssql_python.pybind.connection.connection_pool.cpp: 78.9%
mssql_python.ddbc_bindings.py: 79.6%
mssql_python.pybind.connection.connection.cpp: 81.2%
mssql_python.connection.py: 82.9%
mssql_python.auth.py: 87.1%
mssql_python.pooling.py: 87.7%
mssql_python.helpers.py: 88.9%
mssql_python.__init__.py: 90.7%
mssql_python.exceptions.py: 92.1%

---

🔗 Quick Links

⚙️ Build Summary	📋 Coverage Details
View Azure DevOps Build	Browse Full Coverage Report

[bewithgaurav]

minor comment addition request