FabrizioSandri / RcppDeepState-action

GitHub Action for RcppDeepState
2 stars 0 forks source link

Action logs enhancement #21

Closed FabrizioSandri closed 2 years ago

FabrizioSandri commented 2 years ago

The purpose of this pull request is to improve the action logs when run outside of a pull request. The logs have been reorganized and grouped so that the user may collapse a group and see only the relevant details.

github-actions[bot] commented 2 years ago

RcppDeepState Report

function name message file line address trace R code
rcpp_read_out_of_bound Invalid read of size 4 read_out_of_bound.cpp:7 No Address Trace found
Test code
testlist <- list(rbound = -1203971441L)
result <- do.call(testSAN:::rcpp_read_out_of_bound, testlist)
rcpp_use_after_deallocate Argument 'size' of function __builtin_vec_new has a fishy (possibly negative) value: -1203971441 use_after_deallocate.cpp:6 No Address Trace found
Test code
testlist <- list(array_size = -1203971441L)
result <- do.call(testSAN:::rcpp_use_after_deallocate, testlist)
rcpp_use_after_free Argument 'size' of function malloc has a fishy (possibly negative) value: -4815885764 use_after_free.cpp:6 No Address Trace found
Test code
testlist <- list(alloc_size = -1203971441L)
result <- do.call(testSAN:::rcpp_use_after_free, testlist)
rcpp_write_index_outofbound Invalid write of size 4 write_index_outofbound.cpp:8 No Address Trace found
Test code
testlist <- list(wbound = -1203971441L)
result <- do.call(testSAN:::rcpp_write_index_outofbound, testlist)
rcpp_zero_sized_array Invalid write of size 4 zero_sized_array.cpp:8 zero_sized_array.cpp:7
Test code
testlist <- list(value = -1203971441L)
result <- do.call(testSAN:::rcpp_zero_sized_array, testlist)

Analyzed functions summary

function name tested inputs inputs with issues
rcpp_read_out_of_bound 3 3
rcpp_use_after_deallocate 3 3
rcpp_use_after_free 3 3
rcpp_use_uninitialized 3 0
rcpp_write_index_outofbound 3 3
rcpp_zero_sized_array 3 3

Report details

FabrizioSandri commented 2 years ago

With the last commit I added a branding icon for the action; this will be used on the GitHub marketplace.

tdhock commented 2 years ago

can you post a link to the new output?

FabrizioSandri commented 2 years ago

Here's a link to the latest updated output. As you can see, various parts of the logs(dependencies setup, package installation and analysis) have been grouped together to allow the user to focus on the results. Each group may be extended by clicking on it.

Schermata del 2022-09-06 18-38-09

tdhock commented 2 years ago

great!