On Wed, Nov 09, 2016 at 11:44:11AM +0800, Eryu Guan wrote:
> [sorry, more comments on this patch]
>
> On Fri, Nov 04, 2016 at 05:17:54PM -0700, Darrick J. Wong wrote:
> > Create some routines to help us perform targeted fuzzing of individual
> > fields in various XFS structures. Specifically, we want the caller to
> > drop the xfs_db iocursor on the victim field; from there, the scripts
> > should discover all available fields and fuzzing verbs, and try each
> > fuzz verb on every available field.
> >
> > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> > ---
> > common/fuzzy | 191 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > 1 file changed, 191 insertions(+)
> >
> >
> [snip]
> > +
> > +# Make sure we have all the pieces we need for field fuzzing
> > +_require_scratch_xfs_fuzz_fields()
> > +{
> > + _require_scratch
> > + _require_scrub
> > + _require_populate_commands
> > + _require_command "$XFS_DB_PROG" "xfs_db"
>
> This is included in _require_populate_commands, not needed here.
<nod>
> > + _scratch_mkfs_xfs >/dev/null 2>&1
> > + _scratch_xfs_db -x -c 'fuzz' 2>&1 | grep -q 'not found' && \
> > + _notrun "xfs_db does not have fuzz command"
>
> Does _require_xfs_db_command work here?
Apparently I forgot to update this after inventing _require_xfs_db_command.
Good catch!
--D
>
> Thanks,
> Eryu
>
> > +}
> > +
> > +# Grab the list of available fuzzing verbs
> > +_scratch_xfs_list_fuzz_verbs() {
> > + if [ -n "${SCRATCH_XFS_LIST_FUZZ_VERBS}" ]; then
> > + echo "${SCRATCH_XFS_LIST_FUZZ_VERBS}" | sed -e 's/ /\n/g'
> > + return;
> > + fi
> > + _scratch_xfs_db -x -c 'sb 0' -c 'fuzz' | grep '^Verbs:' | \
> > + sed -e 's/[,.]//g' -e 's/Verbs: //g' -e 's/ /\n/g'
> > +}
> > +
> > +# Fuzz the fields of some piece of metadata
> > +_scratch_xfs_fuzz_fields() {
> > + _scratch_xfs_list_metadata_fields "$@" | while read field; do
> > + _scratch_xfs_list_fuzz_verbs | while read fuzzverb; do
> > + __scratch_xfs_fuzz_mdrestore
> > + __scratch_xfs_fuzz_field_test "${field}" "${fuzzverb}" "$@"
> > + done
> > + done
> > +}
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
