Move and Rename Static Mifare Classic Write Block Function

[pingu2211]

src/mifare/mifarehost.[c/h] implements mfReadBlock and mfReadSector so I was expecting to also find mfWriteBlock and mfWriteSector, but they don't exist. Looking a little further and cmdhfmf.c used a static function static bool mf_write_block for this.
As I want to use it in a hf gallagher extension I'd like it to not be static.

The other similar functions mentioned above return a PM3 status enum rather than bool's, so I also lightly refactored the uses of mfWriteSector to check for PM3_SUCCESS or similar.

I did read

Function names should be separated_with_underscores(), except for standard functions (memcpy, etc.). It may make sense to break this rule for very common, generic functions that look like library functions (e.g. dprintf()).

However the other functions within src/mifare/mifarehost.[c/h] don't follow this convention, and I didn't want this to turn into a major refactor.

[github-actions]

You are welcome to add an entry to the CHANGELOG.md as well

[iceman1001]

Nice,
The current code style is to use all lower case and underscore. I know that the project is filled will all kinds of code styles, but we change it slowly.

So feel free to rename the functions in mifarehost.c accordingly. We will run a make style later once you finished with all your modifications to sort out white space etc.

[iceman1001]

your sector write function will fail if called with a sector number of a MIFARE Classic 4K card.