Page MenuHomeFreeBSD

tests/sys/geom build warning fixes
ClosedPublic

Authored by aprieger_llnw.com on Jun 5 2017, 8:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 10, 12:19 PM
Unknown Object (File)
Feb 18 2024, 3:21 PM
Unknown Object (File)
Jan 22 2024, 8:06 PM
Unknown Object (File)
Jan 1 2024, 3:28 AM
Unknown Object (File)
Dec 31 2023, 4:02 AM
Unknown Object (File)
Dec 25 2023, 8:35 AM
Unknown Object (File)
Nov 18 2023, 4:59 AM
Unknown Object (File)
Nov 8 2023, 7:46 PM
Subscribers

Details

Summary

tests/sys/geom build warning fixes

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9997
Build 10424: arc lint + arc unit

Event Timeline

Changelog:
Pre-fix warnings are at: https://gist.github.com/aprieger-llnw/3fda4ad7d4d0e1a11f6625f796fe8a80

  1. sys/geom/eli/pkcs5v2.c
    • Warnings:
      • warning: passing 'const char *' to parameter of type 'const uint8_t *' (aka 'const unsigned char *') converts between pointers to integer types with different sign [-Wpointer-sign]
      • warning: passing 'uint8_t [11]' to parameter of type 'const char *' converts between pointers to integer types with different sign [-Wpointer-sign]
    • Fixes:
      • added the explicit modifier "const uint8_t *" to parameter "passphrase" to prepare it for the function on line 72
      • added the explicit modifier "const char *" to parameter "passphrase" to prepare it for the function on line 103
sbruno requested changes to this revision.Jun 17 2017, 7:37 PM

Instead of casting to const char *, can you investigate actually changing and/or making the variable type consistent (either const char [] or const uint8_t [])?

This revision now requires changes to proceed.Jun 17 2017, 7:37 PM
aprieger_llnw.com edited edge metadata.
  • tests/sys/geom build warning fixes
  • tests/sys/geom revision update

    Needed to change the variable to const char [], because the change to uint8_t [] would need to cause strcmp to be changed
sbruno added reviewers: oshogbo, pjd.

Should be cosmetic and cleans up warnings only.

This revision is now accepted and ready to land.May 8 2018, 8:43 PM
This revision was automatically updated to reflect the committed changes.