Update: make sure f_new_* calls exist and add f_clear_* calls

Bugfix: string lengths now use signed integers

Update: relocate the shared and static build paths to before the library arguments

Bugfix: firewall show command operating as lock

The debug parameters is conditionally enabled/disabled but the enum is not.
Make the enum conditionally enable/disable the debug option to prevent the wrong id from being used when debug is disabled.

Fix the alignment of the debug message.

Conditionally enable/disable displaying the debug in the help.

Bugfix: add missing closing parenthesis

Bugfix: project compiled libraries are not being found

This is the result of an oversight when I converted the libraries directory to have static and shared into subdirectories.
The explicit link path was not updated accordingly.

Feature: add package generation helper scrript

There are four intended package design modes:
1) individual - each package is separated into their own individual directory and version.
2) level - Each of the three levels (level 0, level 1, and level 2) are put into their own directory and version.
3) monolothic - All three levels (level 0, level 1, and level 2) are put into a single directory and version.

Monolithic is not currently implemented due to design limitations in the generate.sh script and build settings files.

Compression is not performed.

Cleanup: re-organize object file generation into separate directory

The object files should be added in an 'objects/' directory instead of in 'libraray/static/' directory.

Security: use signed integers for string lengths and array lenghts by default

Standard functions, such as strnlen(), appear to operate on signed integers instead of unsigned.
Not being able to handle unsigned integers provides unknown behavior that could lead to potential security vulnerabilities.
Future versions of this project will likely need to abandon these methods for more flexible alternatives.

Example problem:
  sources/c/console.c:36:23: warning: 'strnlen' specified bound 18446744073709551615 exceeds maximum object size 9223372036854775807 [-Wstringop-overflow=]
     36 |       string_length = strnlen(argv[location], f_console_max_size);

That is 2^63 instead of the expected 2^64.

The array lengths were converted to signed as well.

Bugfix: add missing includes

Cleanup: add missing build_static default setting and remove duplicates

Bugfix: variables not being stored in array

Do not directly use string in bash array arguments.
More recent versions of bash do not handle this as expected.
Move explicit string to a variable and then use that variable to access the array.

Progress: add the current (draft) of the custom init program

This is effectively a draft of my ideas and is entirely experimental and incomplete.
I am adding it now because of numerous hard-drive failures and data loss of late.

Update: add new rules to firewall (while fixing accident in previous commit)

The earlier commit that applied stale changes apparently had deleted all the rules in the firewall file.
This data has been added back in addition to the new rules and structure.

Progress: add socket libraries

There is likely more work to do with these libraries.
I will follow this up after reviewing with any changes, if needed.

Progress: miscellaneous changes

I was working on stuff a while back and was taken away from the project for quite a while.
These are some uncommitted changes that I would rather commit immediately and then move forward.

There are some other changes I will follow up in separate commits, such as the socket work and the init work.

Update: add support for separate build_static setting

This allows for building both shared and static libraries and programs.

Update: use firewall_default_allocation_step, +1 was being used and should have been +firewall_default_allocation_step

Update: change default rules for icmp traffic and cleanup some comments

I learned a little bit more about icmp and what is and is not deprecated.
Block certain icmp ports by default.

Shorten the file by removing unnecessary comments.

Update: do not resize strings when length = 0, instead just updated used

I was working on this before I moved.
I am not sure if there is more too this that needs to be done, but for now I will push this.

Bugfix: direction option was confusing because it was mixed with chains

- Separate direction command into two separate commands: direction, and chain.
- The chain command mimics iptables chains and is automatically populated by the custom lists (aka: custom chains).
- When direction is set to something other than none, then the active ethernet device is used.
- Code cleanup and changes were performed while working on this.
- Added basic support for debugging, define _en_firewall_debug_ to enable custom debugging parameter.

Update: add debug as a standard console parameter

Update: oops! don't block unicasts by default

Whoops, I overlooked that unicast was the name for the normal, expected, behavior of standard point to point network connections.

Update: improve the contents firewall settings files

- Add more information to the example firewall settings.
- Improve/fix dhcp client/server rules
- Block/Blackhole unicasts, multicasts, and broadcasts.
- Add some examples rules on conntrack and some security rules.

Bugfix: reverse order of library links

It seems I got the order backwards.
I was using -lc and all the lowest level libraries on the left of the libraries that depend on it.
This caused unobvious linkage errors down the road.
Reversing the order prevented the problem and allowed for static libraries to link.

It is possible the order issue is with ar and not with gcc.

Bugfix: remove more cases where build_alt variables where still being used

Bugfix: fix order of library linkage for firewall build settings

The ar command is sensitive to the order in which libraries are added.
Prior to this change, building a static library could have dependencies used before they are defined due to the order in which libraries were added.
Change the order so that libraries are to the right of their dependencies in the command line argument order

Bugfix: fix order of library linkage

The ar command is sensitive to the order in which libraries are added.
Prior to this change, building a static library could have dependencies used before they are defined due to the order in which libraries were added.
Change the order so that libraries are to the right of their dependencies in the command line argument order.

Bugfix: revert compiler workaround 5bdaa0b, place arguments after source

The problem turns out to be that, for gcc at the very least, linker options such as -lc are not properly passed to the linker when it is before the source files.
Using the examples:
  1) gcc -lc main.c
  2) gcc main.c -lc

In case of #1, the -lc is not passed to the linker.
In case of #2, the -lc is passed to the linker.

All arguments are now being moved after the source parts.

see: 5bdaa0b Workaround: compilers on some systems seem to be sensitive to the order of arguments
see: http://gcc.gnu.org/onlinedocs/gcc/Link-Options.html

Update: f_error changes and more signals

- Added more signal commands.
- Added more signal codes for up to 64 signals.
- Replaced f_error_unmask with f_error_set_fine to be more consistent with the other f_error_set_* names.
- The signal bit is intended to be both the error and warning bit set so it is mathematically the same as the mask bit.
- Change f_none starting point to 197 to handle all possible 64 signals.

Cleanup: explicitly initialize f_false enumerator to 0.

Update: add signal code support to f_errors and move error code starting point to 166

Signals are used as the high order bits in a process that exits, which is where I provide error and similar return codes.
For this reason I am adding wrappers to the x86 signal codes in f_errors instead of a separate custom project such as the non-existant f_signals.

It looks like bash mixes the exit codes and signals into the numbers 129 to 165, so start with 166 as well.

Update: uncomment pipe detection code in return_code and fss_return_code

Not sure why this was disabled, perhaps it was not implemented?
At least, enable the variable.

Feature: add support for -p, --project in generate.sh

This adds support for defining a custom name for the use of preparing and building a project.
The files will be called .built-$project and .prepare-$project.
This allows for building multiple projects in the same directory path back to back.

Cleanup: remove unused parameter 'settings_path'

Update: add level's 0, 1, and 2 settings file to build a single library per level

Update: define f_extern as extern

This should allow for mass changing or mass removing f_extern via a simple custom #define or a command line parameter such as -Df_extern= .

Feature: add support for defining the linker program via build_linker= setting

Bugfix: fix missing dependencies

Cleanup: correct the documentation on the file's level

Bugfix: add missing required library link dependencies

It seems I got away with forgetting some important libraries that were required by some libraries.
This makes me wonder if this is the cause of the bug that forced me to create the build_alt option.

Bugfix: programs incorrectly reporting version

I noticed, unfortunately after the release and not before, that the proper version number was not being reported.
This seems to have been an oversight.

To reduce the chances of this happening, I decided to make the next version number be the default.
I will make a release after a month or two fixing this.

Workaround: compilers on some systems seem to be sensitive to the order of arguments

As far as I can tell, some systems (in my case, it was on linuxmint, either debian or ubuntu) the compiler interprets the following differently:
  1) gcc -lc main.c
  2) gcc main.c -lc

The should be the same, but they are not for some compilers.

This is also not as obvious, some programs or libraries compile while others do not.

In my case, I ran the entire compilation of the f_* fl_* and fll_* libraries using the approach from (1).
Everything compiled and worked, up until I tried to compile my firewall program.
It complained that firewall_main was not found, despite all of the libraries in the path being valid.

I had to recompile the entire f_* fl_* and fll_* libraries using (2) for the firewall program to compile.
No functional changes were made, simply the order of compile arguments.

Therefore, I have implemented an alternative version of the build command, called build_alt.
This should be used if you are using a system where (2) is required over (1).

Bugfix: fix mistake in variable usage within generate.sh

The $sources variable has invalid bash syntax.

Release: FLL 0.4.0

On second thought, 0.3.1 does not need to be released because there is no further development going on in the 0.3.x branch.
Instead, make a stable release (0.4.0).

Release: FLL 0.3.1

Update: remove unavailable script function references

The install and uninstall functions were referenced but are no longer defined.
Just remove their references.

Bugfix: free allocated memory when exiting on fork error

Bugfix: properly initialize name_size string_length

Update: add readme file for generate.sh

Update: add dependencies files

Update: Report total number of content parts when doing a total

This is uneeded for basic and basic list, but extended supports multiple content parts.
When -t is passed while doing a selection using -c, report the total number of content parts instead of the total number found.

Bugfix: correct mistakes in firewall settings files

The files were not entirely synced with my local copies and contained invalid configurations.

Bugfix: fl_execute is now fll_execute

At some point fl_execute and level_1/execute.h where moved to level 2: fll_execute and level_2/execute.h.
This corrects files that did not get updated to reflect this change.

Update: use +v and ++version instead of -v and --version

Use of -v and --version is less common than -h and --help.
Instead, use +v and ++version to free up -v for other purposes like --verbose.

Bugfix: ipv6 icmp is different than ipv4 icmp, adjust rules accordingly

The ipv6 in iptables uses a separate command called icmpv6, with the parameter called icmpv6-type.
Adjust the default settings files to use ipv4 icmp only with iptables and ipv6 icmp only with ip6tables.

This also updates the icmp types for ipv6, adding comments on what each type is.

Bugfix: Fix usage of rule_contents for protocol none option

The if condition that is checking for the existing of the protocol option 'none' was improperly setup.
It was using the rule_objects instead of the rule_contents and therefore would never validate.

Feature: add support for ipv6 firewall via new 'tool' option

This adds the 'tool' option that can assign one of the following values:
- iptables
- ip6tables
- ip46tables

This adds support for ipv6 firewalling.
The default firewall rules enables iptables and ip6tables via the option called ip46tables.

Also, future versions will include support for the tool 'ipset'.

Bugfix: fix build and install data settings file

The settings should be for fll_colors and not fll_print.

Cleanup: move color context structure from level 2 to level 1

Nothing in it depends on level 2.
Move it to level 1, making it available to all level 2 projects.
This renames the structure from fll_color_context to fl_color_context.
Appropriate initializers and memory related macros are renamed accordingly.

Bugfix: fix f_error_is_not_error logic

The logic is wrong.
Just use the error bit instead.
Remove f_error_bit_not_error as it is not needed.

Cleanup: rename/move f_output to f_print

Be consinstent with the project+names wherever possible.
In this case, it is very much possible.
The functions area already called f_print*, so change the name of the project to f_print.

Update: remove *_initialize for specific types

Data types that are numeric, non-pointer, and non-structure should not need an *_initializer macros.
It has been decided for simplicity purposes that these will be left alone.

All other data types that are either non-numeric, pointers, or structures will continue to require *_initialize macros.

Cleanup: remove stale todo notes

These notes were never meant to be committed.
What they represented is already implemented.

Update: return f_unknown at end of f_is_digit

If for some reason the end of the function is reached, f_unknown is returned.
This should be impossible and if compilers give an error, then that line can be removed.

Update: redesign error codes

This is a major rewrite of the error code system.

Previously, the error system did the following:
- used an unsigned short data type
- allowed for all 16 bits to be used
- because it was unsigned short, different architectures might have different size.
- it is rather inefficient and ugly to check for errors when there may be as many as 30 different error codes to compare against.

The new error system does the following:
- enforce exactly 16-bit unsigned integer by using uint16_t.
- reserves the first two high order bits in the following manner:
  - bit 1 = error bit (1000 0000 0000 0000)
  - bit 2 = warning bit (0100 0000 0000 0000)
  - (note setting both bit 1 and bit 2 is unsupported)
- provide a series of macros to quickly detect and manage the new bits
- limits the total available number of error codes to 14-bits instead of 16-bits

The advantage of the new system is that what the status code is can be detected in one or two conditionals (using bitwise operations).
This should speed up performance and make the code easier to read.

Update: add the new default/example firewall setting files

I forgot to add this with one of the previous commits.
This is the rewrite of the setting rules using custom chains.
This also utilizes the newly added protocol support.

Feature: add support for 'protocol' option

The protocol option adds support for mass adding things like '-p tcp' in front of every rule.
This reduces repetition of the -p * commands.

The 'protocol' option accepts any single parameter or the reserved parameter 'none'.
Using 'none' will turn off the protocol option.
Other than 'none', this poses no restrictions on what the protocol may be.
Instead, that is left to iptables.

Update: rewrite firewall, adding support for custom chains

This is a major redesign of the firewall program.
Iptables chains are now supported automatically.
Each new basic list is itself a chain.
Reserved words not allowed for custom chains are:
- main
- stop
- lock

The 'main' chain signifies use of the default iptables chain.

The layout and structure of the files have been rewritten to utilize chains.
The default-firewall file has been removed and replaced with the following files:
- firewall-first
- firewall-last
- firewall-other

The newly written rules defines default firewall rules and chains such that there the following self-explanatory chains:
- input-tcp
- input-udp
- input-icmp
- output-tcp
- output-udp
- output-firewall

In addition to the above changes, the base functionality has been broken up into smaller functions to improve code quality.

Update: add support for array_length arrays

Because this uses memory operations it was moved to a separate file called types_arrays.h
This adds a dependency on f_memory.
While the dependency is circular, f_types will never provide source files.
Therefore chicken and the egg issues should not be a problem (f_types is always installed before f_memory).

Bugfix: fix invalid f_string_lengths initializer

This should initialize to 0 and not the f_string_length_initializer because it is a pointer.

Update: add file position reset macro

Define a macro because creating a function is just overkill for this.
This macro should save a few lines of code when reseting the file position.

Bugfix: reset position on back to back lists

If two lists are back to back such that one does not have content, then make sure that the position resets to the start.
If this does not happen, then the next object will be skipped.

Update: remove used vs size parameter checking

Remove this so that non-dynamic strings can be placed in a dynamic string data type.
In such cases, the dynamic string has a used but not a size.
This means allocations should not be performed on the string.

Cleanup: Move private functions into its own source and header files

It has become painfully apparent that much of the firewall main function needs to be broken down into smaller functions for managability and readability purposes.
The first step in doing such is to create a privte header and source files and move all internal functions there.
Everything in the private-* files are intended to never be exposed in the library.
Its only purpose is to make programming large projects more managable and readable.
As such, none of the functions should ever be considered part of the libraries/programs api.

Bugfix: use proper if then logic in firewall

Some of the if then else statements where not properly nested.
As a result, valid list objects got skipped.

Cleanup: remove stale TODO comment in firewall

The TODO is already done.

Bugfix: remove invalid counter code from firewall

The counter2 was being used when it should not have been.

Update: add support for custom user chains

Users should now be able to add their own custom chains.
Chains can be created by creating a new fss object that is not one of the following: first, last, stop, lock, or main.

Bugfix: fix problems with #defines

Remove an unusued/redundant define.
Use correct size length for some string length defines.

Update: redesign parameters for fss write functions  to be more consistent with fss read functions

The fss read functions all take -o or --object to signify that an object is being processed.
Do the same with the write functions.

Bugfix: set stop point to before a valid close quote, not after

The following reproduces the error:
- fss_basic_write -t object -s 'Hell o\' | fss_basic_read -o
Should produce the original string, but instead included the double quote used to signify object close.

Update: rework delimit management to ensure delimits are only applied when valid.

This unfortunately increases the complexity slightly as deallocation must occur before functions exist.
On the other hand doing so avoids overly complicated loops that would probably make the code less readable.

Bugfix: be sure to delimit trailing \ if it is an the end of a quoted content

What happens is that if the content is (ignore single quotes): 'hell o\'.
The fss extended content conversion should be (include double quotes): "hell o\\".
The code must realize that the last \ is a would end up before a quote, or one would get: "hell o\".
That content string would then be classified as an unterminated group.

Update: add level 3 fss * write projects

Bugfix: Do not de-allocate the additional array

Do not de-allocate this array when some allocation error happens.
The data is defined outside of this function, so leave it to the caller to handle deallocation.
This allows for giving the caller more choices when trying to decided what to do on allocation problem.
For example, some program may want to try to continue anyway even if there is an allocation problem.
It will be tricky, but in such a case one would just have to make do with what is already allocated.

Update: add support for repeated parameters

Thus, something like my_function -s 0 -s 1 would be able to process both -s 0 and -s 1.
Previously, only the last -s option would have been processed.

Bugfix: checking for == on fwrite size results is not an error

The code was checking the total number of bytes written and the total number expected to be written.
If both numbers are the same, then this cannot be an error.

Update: add write append macro

This was missing.

Update: add file write support

The write functions are little more than wrappers to fwrite().

Bugfix: fix positioning output of help system

Update: add fll fss object and content write support

This adds the functions that allow for writing fss objects and contents.

Bugfix: test for eol during object write

If this is not done, then close quotes could end up being placed after a newline character.

Update: add additional error testing macro groups

This adds additional error testing macro groups that are common.

Update: implement fl basic list object and content write support

This adds the fss basic list object and content write functions.

Cleanup: only define location variable when needed

Instead of defining it outside the loop, define it inside the loop and then only in the areas where it is needed.

Update: rewrite basic list object and content read functions

Follow the new approach.
Also, it occured to me that there is no reason to support quoted object names for a basic list.

The reasoning behind this is that the basic list object always terminates on a newline and there cannot be any content on that newline.
If there is no content, then there is no reason to protect the object by wrapping it in quotes as done in basic an extended.
Quote support has been removed and as a result the implementation of the object and content reads are much simpler.

Update: return stop points even when no object/content is found

This makes a things a little more consistent.
If an object was not found, then the code would seek to the end of line or a stop point.
Prior to this patch, after doing the sleep, the appropriate object/content not found is returned when a stop point was reached.
Now, the appropriate no_data or none stop point return value is used in such a case.

Bugfix: fix return state codes for stop point

For objects, if an object was not found and a stop point was reached, return approproate f_no_data codes.
For objects and content, if a stop state was reached during a quoted group, return appropriate f_unterminated_group codes.
For objects and content, if a stop state was reached during a valid pass, return appropriate f_none codes.

Bugfix: don't write comment lines (wrap the # in quotes)

The object write functions should not write comments.
The comments are not valid objects, so if passed a leading #, then be sure to wrap it in quotes.