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

[Bug]: misuse of unsafe vector.GetStringAt #16235

Open
1 task done
XuPeng-SH opened this issue May 20, 2024 · 7 comments
Open
1 task done

[Bug]: misuse of unsafe vector.GetStringAt #16235

XuPeng-SH opened this issue May 20, 2024 · 7 comments
Assignees
Labels
kind/bug Something isn't working severity/s0 Extreme impact: Cause the application to break down and seriously affect the use
Milestone

Comments

@XuPeng-SH
Copy link
Contributor

Is there an existing issue for the same bug?

  • I have checked the existing issues.

Branch Name

main,1.2-dev

Commit ID

latest

Other Environment Information

- Hardware parameters:
- OS type:
- Others:

Actual Behavior

vector.GetStringAt is unsafe and it is misused in some places.

Expected Behavior

No response

Steps to Reproduce

Read code

Additional information

No response

@jensenojs
Copy link
Contributor

@XuPeng-SH 等cp到1.2-dev后徐鹏哥把它直接关掉就可以了吧

@jensenojs jensenojs assigned XuPeng-SH and unassigned matrix-meow May 21, 2024
@XuPeng-SH
Copy link
Contributor Author

@jensenojs 这个实际上是很多模块owner要做的,我那个pr是把几个明显的问题先解决掉

@XuPeng-SH XuPeng-SH assigned ouyuanning and unassigned XuPeng-SH May 22, 2024
@XuPeng-SH
Copy link
Contributor Author

@ouyuanning please investigate this problem in your modules

@ouyuanning
Copy link
Contributor

还没时间看

@sukki37 sukki37 modified the milestones: 1.2.0, 1.2.1 May 28, 2024
@ouyuanning
Copy link
Contributor

在处理w-w和prepare,还没时间处理

@ouyuanning
Copy link
Contributor

这周处理

@ouyuanning
Copy link
Contributor

还没时间清理

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working severity/s0 Extreme impact: Cause the application to break down and seriously affect the use
Projects
None yet
Development

No branches or pull requests

6 participants