Skip to content
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

fix offset>=so.size() assertion #1272

Merged
merged 3 commits into from
Dec 6, 2023
Merged

fix offset>=so.size() assertion #1272

merged 3 commits into from
Dec 6, 2023

Conversation

bjjwwang
Copy link
Contributor

@bjjwwang bjjwwang commented Dec 6, 2023

if offset is larger than so.size(), set offset the last element.

Copy link

codecov bot commented Dec 6, 2023

Codecov Report

Merging #1272 (0ea3137) into master (1ec4620) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

❗ Current head 0ea3137 differs from pull request most recent head b40866a. Consider uploading reports for the commit b40866a to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1272      +/-   ##
==========================================
- Coverage   64.50%   64.49%   -0.02%     
==========================================
  Files         223      223              
  Lines       23796    23800       +4     
==========================================
  Hits        15349    15349              
- Misses       8447     8451       +4     
Files Coverage Δ
svf/lib/MemoryModel/AccessPath.cpp 3.12% <0.00%> (-0.14%) ⬇️

@yuleisui yuleisui merged commit 2b3b9f8 into SVF-tools:master Dec 6, 2023
3 checks passed
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 this pull request may close these issues.

None yet

2 participants