Fixed unsafe string copy and concat in `fileio.c`.

Per warnings from flawfinder: "Does not check for buffer overflows when
copying to destination [MS-banned] (CWE-120). Consider using snprintf,
strcpy_s, or strlcpy (warning: strncpy easily misused).".

Replaced called to strcpy and strcat in `fileio.c` to calls with a
specified size (`strncpy` and `strncat`).

Tested the changes on OSX, Linux, Windows.
On OSX + Linux, changes were tested with ASAN. The following flags were
used: 'check_initialization_order=1:strict_init_order=1:detect_odr_violation=1:detect_stack_use_after_return=1'

To reproduce warning:
./flawfinder.py ./programs/fileio.c

Merge pull request #1236 from GeorgeLu97/paramgrillconstraints

ParamgrillConstraints

Default lvl 1

MB to 2^20

Reorder declaration

fixed assert() condition

minor code comments improvements

Merge pull request #1273 from facebook/zstdgrep

added zstdgrep documentation

added zstdgrep documentation

presenting `zstdgrep` limit regarding dictionary compression
with workaround recommended by @tobwen (#1268)

Remove ctx from benchMem

Add consts

+ fix gcc-8 warnings

argument parsing cleanup

+ clarifying comment

Readability improvements, renaming

const srcPtrs

Change speed representation from floating point to integral

Update Documentation

Change comment // to  /* */
Add more description of what functions do
Remove outdated comments

Renaming / Style fixes

Separate capacity vs size

Also:
Make suggested fixes
-varInds_t
-reorder some arguments
-remove code duplication
-update README / -h
-Fix memory leaks

Fix windows implicit casting bugs

Add Levels

BenchMem with block compressed sizes passed back up

Total Changes:

Add different constraint types (decompression speed, compression memory, parameter constraints)
Separate search space by strategy + strategy selection
Memoize results
Real random restarts
Support multiple files
Support Dictionary inputs
Debug Macro for extra printing

Dictionary + Multiple file Loading

Not crashing

Working V1

Parses additional parameters

Additional constraint checking

Minor fixes

more param parsing

Add Memory

Change paramVariation

work on feasibility

reformat bench

Changed Paramgrill to use bench.c benchmarking

customlevel macro

Printing Flag

Minor changes

Explicit casting

Makefile fix

casting, type fix

Printing Flag

Minor Changes

comments, helper fn's

Merge pull request #1272 from facebook/travisTest

gcc-8 travis test

Merge pull request #1271 from felixhandte/circleci-2.0-config

Migrate Config to CircleCI 2.0

Preserve Commented-Out Longer Tests

some errors may only happen when optimizations are enabled

Remove Old CircleCI Config

Fix Path

Add an Auto-Gen'ed CircleCI 2.0 Config

Built via the cci-config-generator.sh script.

try adding gcc-8 compilation tests

try to improve some travis test speed

Merge pull request #1243 from facebook/fix1241

fix #1241

Merge pull request #1269 from felixhandte/fix-fix1241

Fix fix1241: Fix CDict Attachment to Handle CDicts with Non-Zero Starts

Fix CDict Attachment to Handle CDicts with Non-Zero Starts

CDicts were previously guaranteed to be generated with `lowLimit=dictLimit=0`.
This is no longer true, and so the old length and index calculations are no
longer valid. This diff fixes them to handle non-zero start indices in CDicts.

Merge branch 'dev' into fix1241

Merge pull request #1263 from facebook/travisTest

Fix Makefile on Linux and ZSTD_NOBENCH

fix unused variable warning

fix .travis.yml

fix : zstd cli can be built with build macro ZSTD_NOBENCH

which disables bench.c module

Merge branch 'dev' into travisTest

Merge branch 'travisTest' of github.com:facebook/zstd into travisTest

Merge pull request #1255 from terrelln/norm-fix

[FSE] Fix division by zero

Disallow tableLog == srcLog

Add non-optimize FASTCOVER (#1260)

* Add non-optimize FASTCOVER

* Minor fix

* Pass param as value instead of pointer

Merge pull request #1258 from jennifermliu/fastCover

Use CDict instead of CCtx in analyzeEntropy

Refactoring

Use CDict instead of CCtx in analyzeEntropy

Merge pull request #1257 from jennifermliu/fastCover

Increment frequency for every dmer occurence within same sample

[FSE] Add division by zero test

Increment frequency for every dmer occurence within same sample instead of at most once per sample

speed up appveyor tests

fix %zu support under minGW

and relevant test on Appveyor

ensure appveyor test fails due to formatting error

to catch %zu incompatibility

Merge branch 'dev' into fix1241

support %zu under mingw

Merge pull request #1254 from jennifermliu/fastCover

 Save segmentFreqs in ctx instead of malloc and memset in SelectSegment

Delete old benchmarking result

[FSE] Fix division by zero

When the primary normalization method fails, and
`(1 << tableLog) == (maxSymbolValue + 1)`, and every symbol gets assigned
normalized weight 1 or -1 in the first loop, then the next division can
raise `SIGFPE`.

Undo deleting clean in make

Revert test.sh

Save segmentFreqs in ctx instead of malloc and memset in SelectSegment

Merge pull request #1250 from jennifermliu/fastCover

Add Fast Cover Dictionary Builder

Use same param after optimizing cover and fastCover and record k and d for benchmarking

Rerun cover and fastCover with optimized values

Benchmark different f values

Merge pull request #1252 from GeorgeLu97/bench-quiet-display-fix

Fixes bug

no zero

Add array to keep track of frequency within active segment, fix malloc bug, update benchmarking result

Allow d=6

Benchmark fast cover optimize vs k=200

Initial benchmarking result for fastCover

Make hash value const

Add Fast Cover Dictionary Builder

Merge pull request #1246 from jennifermliu/benchmark

Benchmark dictionary builders

Minor fix

Refactoring and benchmark without dictionary

Benchmark dictionary builders

Merge pull request #1245 from terrelln/disable-arm

Revert "attempt to re-enable arm64 tests"

Revert "attempt to re-enable arm64 tests"

This reverts commit 9c277f137cbcaa385ff5b95ec4cbdce50675541d.

Merge pull request #1238 from jennifermliu/random

Add random dictionary builder

Update Makefile

Rename cleanup

Add cleanup to trainfromFiles and move RANDOM_segment_t declaration

Merge pull request #1230 from terrelln/train-out

zstdcli: Allow -o before --train

Merge pull request #1237 from terrelln/init-cstream-adv

Set requestedParams in ZSTD_initCStream*()

fix minor conversion warning

Merge pull request #1242 from facebook/mingw_traces

zstdmt debug traces compatibles with mingw

Merge pull request #1244 from terrelln/ferror

fileio: Error in compression on read errors

Update freeSampleInfo

Fix deallocation problem and add documentation

fileio: Error in compression on read errors

We can write a corrupted file if the input file errors during a read.
We should return a non-zero error code in this case.

Make test PHONY target

Move file loading functions to new file for access by benchmarking tool

fix make test on Linux

MOREFLAGS+= doesn't work on Linux

added a test to be played on travis

to check a make variable construction

fix #1241

Ensure that first input position is valid for a match
even during first usage of context
by starting reference at 1
(avoiding the problematic 0).

zstdmt debug traces compatibles with mingw

since mingw does not have `sys/times.h`,
remove this path when detecting mingw compilation.