Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1e4b6842 authored by Henri Chataing's avatar Henri Chataing
Browse files

system/stack/test/fuzzers: Enforce -Wunused-parameter 

Bug: 299772495
Test: m com.android.btservices
Flag: EXEMPT, mechanical refactor
Change-Id: I754d1bc2f0ddfb476b1a32b5964245458385a64f
parent f097560a

system/stack/test/fuzzers/a2dp/Android.bp

+0 −1
Original line number Diff line number Diff line
@@ -14,5 +14,4 @@ cc_fuzz { 
    srcs: [

        "fuzz_a2dp.cc",

    ],

    cflags: ["-Wno-unused-parameter"],

}

system/stack/test/fuzzers/a2dp/a2dpFuzzHelpers.h

+2 −1
Original line number Diff line number Diff line
@@ -44,7 +44,8 @@ tA2DP_SDP_DB_PARAMS generateDBParams(FuzzedDataProvider* fdp, std::vector<uint16 
}



// Define our empty callback function

void a2dp_find_callback(bool found, tA2DP_Service* p_service, const RawAddress& peer_address) {

void a2dp_find_callback(bool /*found*/, tA2DP_Service* /*p_service*/,

                        const RawAddress& peer_address) {

  // Free the RawAddress we created in the generate function

  delete &peer_address;

}

system/stack/test/fuzzers/a2dp/codec/Android.bp

+0 −3
Original line number Diff line number Diff line
@@ -24,7 +24,6 @@ cc_fuzz { 
    srcs: [

        "fuzz_a2dp_codec.cc",

    ],

    cflags: ["-Wno-unused-parameter"],

}



cc_fuzz {
@@ -34,7 +33,6 @@ cc_fuzz { 
    srcs: [

        "fuzz_a2dp_codec_info.cc",

    ],

    cflags: ["-Wno-unused-parameter"],

}



cc_fuzz {
@@ -44,5 +42,4 @@ cc_fuzz { 
    srcs: [

        "fuzz_a2dp_codec_config.cc",

    ],

    cflags: ["-Wno-unused-parameter"],

}

system/stack/test/fuzzers/a2dp/codec/a2dpCodecInfoFuzzFunctions.h

+27 −19
Original line number Diff line number Diff line
@@ -36,7 +36,7 @@ 
 */

std::vector<std::function<void(FuzzedDataProvider*, uint8_t*)>> a2dp_codec_info_operations = {

        // A2DP_InitDefaultCodec

        [](FuzzedDataProvider* fdp, uint8_t*) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t*) -> void {

          // Allocate space for a new codec & add it to our tracking vector

          uint8_t* codec_info = new uint8_t[AVDT_CODEC_SIZE];

          a2dp_codec_info_vect.push_back(codec_info);
@@ -58,25 +58,27 @@ std::vector<std::function<void(FuzzedDataProvider*, uint8_t*)>> a2dp_codec_info_ 
        },



        // A2DP_GetCodecType

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void { A2DP_GetCodecType(codec_info); },

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetCodecType(codec_info);

        },



        // A2DP_IsSourceCodecValid

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_IsSourceCodecValid(codec_info);

        },



        // A2DP_IsPeerSourceCodecValid

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_IsPeerSourceCodecValid(codec_info);

        },



        // A2DP_IsPeerSinkCodecValid

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_IsPeerSinkCodecValid(codec_info);

        },



        // A2DP_IsSinkCodecSupported

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_IsSinkCodecSupported(codec_info);

        },
@@ -86,10 +88,14 @@ std::vector<std::function<void(FuzzedDataProvider*, uint8_t*)>> a2dp_codec_info_ 
        },



        // A2DP_GetMediaType

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void { A2DP_GetMediaType(codec_info); },

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetMediaType(codec_info);

        },



        // A2DP_CodecName

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void { A2DP_CodecName(codec_info); },

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_CodecName(codec_info);

        },



        // A2DP_CodecTypeEquals

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {
@@ -108,22 +114,22 @@ std::vector<std::function<void(FuzzedDataProvider*, uint8_t*)>> a2dp_codec_info_ 
        },



        // A2DP_GetTrackSampleRate

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetTrackSampleRate(codec_info);

        },



        // A2DP_GetTrackBitsPerSample

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetTrackBitsPerSample(codec_info);

        },



        // A2DP_GetTrackChannelCount

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetTrackChannelCount(codec_info);

        },



        // A2DP_GetSinkTrackChannelType

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetSinkTrackChannelType(codec_info);

        },
@@ -152,25 +158,27 @@ std::vector<std::function<void(FuzzedDataProvider*, uint8_t*)>> a2dp_codec_info_ 
        },



        // A2DP_GetEncoderInterface

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetEncoderInterface(codec_info);

        },



        // A2DP_GetDecoderInterface

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetDecoderInterface(codec_info);

        },



        // A2DP_AdjustCodec

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void { A2DP_AdjustCodec(codec_info); },

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_AdjustCodec(codec_info);

        },



        // A2DP_SourceCodecIndex

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_SourceCodecIndex(codec_info);

        },



        // A2DP_SinkCodecIndex

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_SinkCodecIndex(codec_info);

        },
@@ -186,12 +194,12 @@ std::vector<std::function<void(FuzzedDataProvider*, uint8_t*)>> a2dp_codec_info_ 
        },



        // A2DP_GetEecoderEffectiveFrameSize

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_GetEecoderEffectiveFrameSize(codec_info);

        },



        // A2DP_CodecInfoString

        [](FuzzedDataProvider* fdp, uint8_t* codec_info) -> void {

        [](FuzzedDataProvider* /*fdp*/, uint8_t* codec_info) -> void {

          A2DP_CodecInfoString(codec_info);

        }};

system/stack/test/fuzzers/sdp/Android.bp

+0 −1
Original line number Diff line number Diff line
@@ -14,5 +14,4 @@ cc_fuzz { 
    srcs: [

        "fuzz_sdp.cc",

    ],

    cflags: ["-Wno-unused-parameter"],

}