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.

Sign up for GitHub

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

Jump to bottom

lightning: fix risk of OOM (#40443) #44333

Merged
merged 2 commits into from
Jun 1, 2023
Merged

lightning: fix risk of OOM (#40443) #44333

merged 2 commits into from
Jun 1, 2023

Commits on Jun 1, 2023

  1. lightning: not set pos to end if fail to ReadUntil (pingcap#40232) 

    * fix: not set pos to end

* add it

* fix: return to prevPos

* fix: return err

* fix

* fix

* fix it

* fix

* fix: not exit when err

Co-authored-by: Zak Zhao <57036248+joccau@users.noreply.github.com>
    @buchuitoudegou @joccau @lance6716
    2 people authored and lance6716 committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    fb3cf4e View commit details
    Browse the repository at this point in the history

  2. lightning: fix risk of OOM (pingcap#40443) 

    * fix: check buf size

* fix: terminate too long buf

* add ut

* fix dev

* local var

* fix ut

* fix lint

* fix lint

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
    @buchuitoudegou @ti-chi-bot @lance6716
    2 people authored and lance6716 committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    bf8c7b5 View commit details
    Browse the repository at this point in the history