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

[networkanalyzer]Subscribe sendmmsg to parse DNS requests #430

Merged
merged 13 commits into from
Feb 24, 2023
Merged

[networkanalyzer]Subscribe sendmmsg to parse DNS requests #430

merged 13 commits into from
Feb 24, 2023

Conversation

dxsup
Copy link
Member

@dxsup dxsup commented Jan 4, 2023 •
edited
Loading

Description

Subscribe sendmmsg to parse more DNS requests.

Related Issue

#429

How Has This Been Tested?

Have added a new test case TestDnsProtocol/sendmmsg to cover this issue.

=== RUN   TestDnsProtocol
=== RUN   TestDnsProtocol/multi
=== RUN   TestDnsProtocol/sendmmsg
--- PASS: TestDnsProtocol (0.00s)
    --- PASS: TestDnsProtocol/multi (0.00s)
    --- PASS: TestDnsProtocol/sendmmsg (0.00s)
PASS

Subscribe sendmmsg and recvmmsg
12f41b0 
Signed-off-by: Daxin Wang <daxinwang@harmonycloud.cn>
@dxsup dxsup requested a review from hocktea214 January 4, 2023 07:22
Update the CHANGELOG.md
0085c7f 
Signed-off-by: Daxin Wang <daxinwang@harmonycloud.cn>
hocktea214
hocktea214 previously approved these changes Jan 4, 2023
View reviewed changes
@dxsup dxsup dismissed hocktea214’s stale review January 4, 2023 09:12

The data in Sendmmsg is empty. The probe should first handle it.

@dxsup dxsup marked this pull request as draft January 4, 2023 09:38
sanyangji and others added 6 commits January 12, 2023 15:31
@sanyangji
enhance sendmmsg
fdb9c8c 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@dxsup
Merge pull request #1 from sanyangji/add-sendmmsg-event
0c8fd31 
enhance sendmmsg
Merge branch 'main' into add-sendmmsg-event
bb81b93 
# Conflicts:
#	CHANGELOG.md
Rename the receiver name of KindlingEvent to make it consistent
f2063dd 
Signed-off-by: Daxin Wang <daxinwang@harmonycloud.cn>
Analyze the syscall sendmmsg
40fe120 
Signed-off-by: Daxin Wang <daxinwang@harmonycloud.cn>
Analyze the syscall sendmmsg
43040b2 
Signed-off-by: Daxin Wang <daxinwang@harmonycloud.cn>
@dxsup dxsup marked this pull request as ready for review February 22, 2023 09:52
@dxsup dxsup requested a review from NeJan2020 February 22, 2023 09:52
Update the CHANGELOG.md
97a9187 
Signed-off-by: Daxin Wang <daxinwang@harmonycloud.cn>
@dxsup dxsup changed the title [networkanalyzer]Subscribe sendmmsg and recvmmsg to parse DNS requests [networkanalyzer]Subscribe sendmmsg to parse DNS requests Feb 22, 2023
hocktea214
hocktea214 previously approved these changes Feb 23, 2023
View reviewed changes
@sanyangji
update agent-libs
29b118a 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@dxsup dxsup requested a review from hocktea214 February 24, 2023 02:16
hocktea214
hocktea214 previously approved these changes Feb 24, 2023
View reviewed changes
@sanyangji
update agent-libs
122094d 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@sanyangji
update agent-libs
9fdc655 
Signed-off-by: sanyangji <songyujie@zju.edu.cn>
@dxsup dxsup requested a review from hocktea214 February 24, 2023 03:46
hocktea214
hocktea214 previously approved these changes Feb 24, 2023
View reviewed changes
@dxsup dxsup merged commit 0a32ce6 into KindlingProject:main Feb 24, 2023
@dxsup dxsup deleted the add-sendmmsg-event branch February 24, 2023 03:55
@dxsup dxsup linked an issue Feb 24, 2023 that may be closed by this pull request
Can't analyze DNS requests with the sendmmsg syscall #429
Closed
This was referenced Feb 24, 2023
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hocktea214 hocktea214 hocktea214 approved these changes

@NeJan2020 NeJan2020 Awaiting requested review from NeJan2020

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Can't analyze DNS requests with the sendmmsg syscall
3 participants
@dxsup @hocktea214 @sanyangji