Skip to content

Add support to filter on exception message#160

Merged
MarioHewardt merged 9 commits intomicrosoft:masterfrom
oldzhu:filtermsg
Feb 24, 2023
Merged

Add support to filter on exception message#160
MarioHewardt merged 9 commits intomicrosoft:masterfrom
oldzhu:filtermsg

Conversation

@oldzhu
Copy link
Copy Markdown
Contributor

@oldzhu oldzhu commented Feb 3, 2023

Add support to filter on exception message and make a dump file name contains the exception name and dump count.
Resolves #159, Resolves #161

1. Add the new function CorProfiler::GetExceptionMessage
2. Add filter on exception message in CorProfiler::ExceptionThrown
3. Make a core dump file name contains exception name and dump count
4. Modify PrintUsage, man page and README.md to reflect the filter change

Signed-off-by: oldzhu <wp_xdzhu@hotmail.com>
… the dump count,

modify the testing scripts which try to locate the exception dumps by name accordingly.

Signed-off-by: oldzhu <wp_xdzhu@hotmail.com>
…ocdump in the related scripts

2. Add -log option in all procdump command running in the testing scripts for troubleshooting failed testing cases
3. Add sleep betwen the procdump and wget to trigger exception so that procdump is ready for exception before exception thrown
4. Add access call loop checking in profiler to make sure procdump listening on the socket before profiler try to connect

Signed-off-by: oldzhu <wp_xdzhu@hotmail.com>
Comment thread profiler/src/ProcDumpProfiler.cpp Outdated
Comment thread profiler/src/ProcDumpProfiler.cpp
Comment thread profiler/src/ProcDumpProfiler.cpp Outdated
Comment thread profiler/src/ProcDumpProfiler.cpp Outdated
Comment thread tests/integration/scenarios/dotnet_1FilterExceptionMsgDump1Thrown_dump.sh Outdated
…f a procdump socket is created or not

2. fix medta leak in CorProfiler::GetExceptionMessage while loop
4. change delete to delete [] in CorProfiler::GetExceptionMessage
3. limit socket access testing to 5 times to avoid endless loop

Signed-off-by: oldzhu <wp_xdzhu@hotmail.com>
@MarioHewardt MarioHewardt merged commit 85e46f9 into microsoft:master Feb 24, 2023
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.

failed to collect an exception dump as profiler unloaded itself due to failed to connect to procdump adding support to filter on exception message

2 participants