Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support for eBPF based port forwarding #3067

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

balajiv113
Copy link
Member

@balajiv113 balajiv113 commented Dec 30, 2024

Improve port forwarding by using ebpf

Note: Test run. Locally it works. Wanted to confirm with all jobs

ev := &api.Event{Time: timestamppb.Now()}
atoi, _ := strconv.Atoi(event.Port)
if event.Action == trackapi.OPEN {
port[0] = &api.IPPort{Ip: event.Ip.String(), Port: int32(atoi), Protocol: trackapi.ProtocolToString(event.Protocol)}

Check failure

Code scanning / CodeQL

Incorrect conversion between integer types High

Incorrect conversion of an integer with architecture-dependent bit size from
strconv.Atoi
to a lower bit size type int32 without an upper bound check.
port[0] = &api.IPPort{Ip: event.Ip.String(), Port: int32(atoi), Protocol: trackapi.ProtocolToString(event.Protocol)}
ev.LocalPortsAdded = port
} else {
port[0] = &api.IPPort{Ip: event.Ip.String(), Port: int32(atoi), Protocol: trackapi.ProtocolToString(event.Protocol)}

Check failure

Code scanning / CodeQL

Incorrect conversion between integer types High

Incorrect conversion of an integer with architecture-dependent bit size from
strconv.Atoi
to a lower bit size type int32 without an upper bound check.

Verified

This commit was signed with the committer’s verified signature.
renovate-bot Mend Renovate
Signed-off-by: Balaji Vijayakumar <kuttibalaji.v6@gmail.com>
@AkihiroSuda
Copy link
Member

Needs rebase

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants