]> Kevux Git Server - fll/log
fll
2 years agoBugfix: Memory handling related to or exposed by f_directory unit tests.
Kevin Day [Sat, 9 Apr 2022 05:42:10 +0000 (00:42 -0500)]
Bugfix: Memory handling related to or exposed by f_directory unit tests.

Replace a double pointer cast with a single pointer cast.

Just in case, if the struct dirent listing is allocated, free it when scandir() returns -1.

The entire struct dirent pointers do not need to be allocated as I thought.
It seems these are copied inside scandir() where the memory deletes inside that function did not affect the mock pointer from the caller.
This resulted in a minor memory leak in the unit test program.
Replace the outermost struct dirent malloc with just a struct dirent array in these cases.

2 years agoUpdate: Fix the f_directory unit tests and finish writing them.
Kevin Day [Sat, 9 Apr 2022 05:25:03 +0000 (00:25 -0500)]
Update: Fix the f_directory unit tests and finish writing them.

The problem with the unit tests not being run (especially on Github) appears to be the result of compiler optimizations.
Fix this by disabling compiler optimizations in the mock functions and add appropriate commetns explaining this.

Do not wrap/mock strncmp() and strnlen().

The directory tests are now completed.

2 years agoBugfix: Invalid memory deletion.
Kevin Day [Sat, 9 Apr 2022 05:18:25 +0000 (00:18 -0500)]
Bugfix: Invalid memory deletion.

The memory is being incorrectly deleted for multiple reasons.
1) The type is not f_string_t, it is struct dirent.
2) The index within a listing is already a pointer.
3) The listing itself is already a pointer.
4) Remove extra delete commands.
5) The memory is not being deleted on certain errors.

These problems have been exposed by the unit tests.

Also replace macro_f_memory_structure_increment() with f_string_dynamics_increase().

2 years agoUpdate: Settings, mock settings, and test settings.
Kevin Day [Sat, 9 Apr 2022 03:46:24 +0000 (22:46 -0500)]
Update: Settings, mock settings, and test settings.

Add standard comments to the settings-mocks and settings-tests files.

Remove the level and monolithic from the settings-mocks and settings-tests files given that I have now determined to rrun the tests exclusively in individual build mode.

Minor clean ups in these files.

2 years agoUpdate: Use off_t instead of f_array_length_t to match uses in functions like f_file_...
Kevin Day [Fri, 8 Apr 2022 03:44:11 +0000 (22:44 -0500)]
Update: Use off_t instead of f_array_length_t to match uses in functions like f_file_size().

2 years agoUpdate: Simplify build settings files.
Kevin Day [Fri, 8 Apr 2022 03:35:41 +0000 (22:35 -0500)]
Update: Simplify build settings files.

The build setting files contained numerous Objects set to empty content.
This allows for showing how to configure.

With a long standing bug fixed (defaults being applied incorrectly) a misconfiguration as a result of this has been revealed.

Simplify the build settings to make the settings easier to read and maintain.
This ends up fixing the bug (because the default path_sources gets assigned when path_sources is not defined at all).

Do some minor structural cleanup in these files.

2 years agoCleanup: Add missing closing quote.
Kevin Day [Fri, 8 Apr 2022 01:20:43 +0000 (20:20 -0500)]
Cleanup: Add missing closing quote.

2 years agoUpdate: The fake program should show the fakefile or setting being used when building.
Kevin Day [Fri, 8 Apr 2022 01:18:52 +0000 (20:18 -0500)]
Update: The fake program should show the fakefile or setting being used when building.

This helps make what is going more clear when viewing the console.
Multiple executions of different build settings (or fakefiles) should be easily determinable such as when running tests.

2 years agoCleanup: Code structure for fake.
Kevin Day [Fri, 8 Apr 2022 01:13:39 +0000 (20:13 -0500)]
Cleanup: Code structure for fake.

2 years agoUpdate: Add new return status codes: F_debug, F_error, F_fine, and F_warning (again).
Kevin Day [Thu, 7 Apr 2022 06:18:32 +0000 (01:18 -0500)]
Update: Add new return status codes: F_debug, F_error, F_fine, and F_warning (again).

I missed a couple of files with the "git add" command.

Also add handling of F_error to the standard error handler.

2 years agoUpdate: Add new return status codes: F_debug, F_error, F_fine, and F_warning.
Kevin Day [Thu, 7 Apr 2022 06:11:20 +0000 (01:11 -0500)]
Update: Add new return status codes: F_debug, F_error, F_fine, and F_warning.

Also add handling of F_error to the standard error handler.

2 years agoCleanup: Use proper sentence casing in comment.
Kevin Day [Thu, 7 Apr 2022 05:54:08 +0000 (00:54 -0500)]
Cleanup: Use proper sentence casing in comment.

2 years agoUpdate: Improve read program logic to allow for catching interrupts during file read.
Kevin Day [Thu, 7 Apr 2022 05:49:09 +0000 (00:49 -0500)]
Update: Improve read program logic to allow for catching interrupts during file read.

Introduce break points in the read process by establishing an outer block loop.
Add the signal check before each block read.
There may need to be additional, smaller checks to further reduce how often the signal check is performed within these loops.

Make sure to use the file stream read rather than the file read.

Calculate the file size and pre-allocate the buffer.
Determine the exact size needed to avoid additional reallocations by utilizing the modulus operator.

The fss_embedded_read is notably out of day and inconsistent with the rest of the fss read programs.
This is only partially updated enough to work with the changes.

2 years agoBugfix: For f_file, use off_t, pre-test for EOF or error, use sizeof(f_char_t), and...
Kevin Day [Thu, 7 Apr 2022 05:31:34 +0000 (00:31 -0500)]
Bugfix: For f_file, use off_t, pre-test for EOF or error, use sizeof(f_char_t), and use size_t rather than ssize_t.

The fread() and similar stream functions actually use size_t rather than ssize_t.
Fix this bug by changing ssize_to to size_to for the affected functions.

Use off_t rather than f_array_length_t to be consistent with the types used in the libc/POSIX API.

Add additional checks for feof() and ferror() and the start of the stream read functions.
There needs to be a few more error codes, add a TODO comment to address this via a separate commit.

Use the size of f_char_t rather than 1 for reading.

Use >= in buffer read check rather than ==, just in case.

2 years agoUpdate: Remove extra error print in fss_embedded_list_read.
Kevin Day [Thu, 7 Apr 2022 01:23:57 +0000 (20:23 -0500)]
Update: Remove extra error print in fss_embedded_list_read.

2 years agoUpdate: The example project build settings and fakefiles.
Kevin Day [Wed, 6 Apr 2022 03:37:30 +0000 (22:37 -0500)]
Update: The example project build settings and fakefiles.

These fell out of sync.
Add support for building via clang.
The "example-objects" project doesn't include support for clang because it is providing a minimalistic example.

The "path_standard" is now "has_path_standard".

2 years agoBugfix: Defaults should not be applied when an empty Object is specified.
Kevin Day [Wed, 6 Apr 2022 03:32:43 +0000 (22:32 -0500)]
Bugfix: Defaults should not be applied when an empty Object is specified.

When an Object is not specified, such as with "path_sources", then the default should be loaded.
When an Object is specified but ha Content with a length of 0, then the Content should not have a default applied.
The example bzip fakefile exposed this problem where the "path_sources" is empty and should be.
The existing code is incorrectly applying the default when length is 0.

The code to actually check for whether or not an Object is specified is already in place but it is not used.
Update the code to utilize the existing information.

2 years agoCleanup: Remove old todo comment.
Kevin Day [Wed, 6 Apr 2022 03:32:20 +0000 (22:32 -0500)]
Cleanup: Remove old todo comment.

2 years agoBugfix: The f_capability should accept f_string_constant_t rather than f_string_static_t.
Kevin Day [Tue, 5 Apr 2022 03:35:54 +0000 (22:35 -0500)]
Bugfix: The f_capability should accept f_string_constant_t rather than f_string_static_t.

The name.string is incorrect and should be name->string.
Once I fixed this, I noticed a compiler warning about losing the "const" frrom the return value of cap_mode_name().
Use f_string_constant_t rather than f_string_static_t.

2 years agoCleanup: Remove extra line.
Kevin Day [Tue, 5 Apr 2022 03:35:28 +0000 (22:35 -0500)]
Cleanup: Remove extra line.

2 years agoUpdate: Add f_directory tests then fix problems discovered problems and perform appro... 0.5.9
Kevin Day [Sun, 3 Apr 2022 15:59:05 +0000 (10:59 -0500)]
Update: Add f_directory tests then fix problems discovered problems and perform appropriate cleanups.

The "preserve_path_headers" in settings-mocks and settings-tests should be set to "yes".

Add missing dependencies-tests file for any project providing tests.

Remove at_id <= 0 parameter checks because this is already handled by EBADF.

Fix alphabetic ordering of errno returns for f_directory.
I started doing this for f_file as well until I realized how much there is to do.
I won't continue this for f_file because I don't plan on adding tests for f_file for this next release.

The ENOTDIR should return F_directory_not rather than F_directory.

Unable to perform many of the tests because the dynamic linker wrapper is not properlly handling stat() and scandir() as expected.
These tests are commented out, incomplete, and comments are added describing why.

2 years agoBugfix: UTF-8 endianess tests fail and improve seek to functions for UTF-8.
Kevin Day [Sun, 3 Apr 2022 00:41:34 +0000 (19:41 -0500)]
Bugfix: UTF-8 endianess tests fail and improve seek to functions for UTF-8.

The original commit dda60cc8d9cad69b55e7d79b4c052677b03b4413 introduced reversed endianness logic.
The tests passed for this and so the tests were wrong.

The commit that fixed the reversed endianness problems is 78daa0d760fbcc767dd099d68ae02d23b3df4991.
These changes revealed the test problems due to the tests now failing.

Fix the tests.
Make a new character type for the EOL, EOS, and placeholder in a way that is more consistent with the character macros for the f_char_t.

The F_utf_substitute_open_box_d_length and F_utf_substitute_open_box_shouldered_d_length should have _s_ rather than _d_ in their names.

Change the UTF-8 seek to functions to not need to check for URL fragments.
Change the UTF-8 seek to functions to accept f_utf_char_t rather than f_char_t.
This completely makes more sense.
This allows for searching a f_utf_string_t using a f_utf_char_t.
The tests are expecting this behavior earlier and they only happened to succeed due to the coincidence of the bitwise order of the previously incorrect tests.

This swaps the bitwise order of the strings.
More than likely I need a macro check to handle big endian as well.
This is left to changes in the future.
The 0.6.x stable series does not guarantee big endian support.
The 0.8.x stable series does guarantee big endian support because I will be working on adding networking support.
Leave the bitwise improvements and tests for the next development cycle so that I can focus on pushing out the 0.6.x stable release.
If I have the time, I might still fix this after the release candidate is released as I will most likely be improving the tests and fixing any discovered problems.

2 years agoUpdate: Get the controller program in a better state for a stable release.
Kevin Day [Sat, 2 Apr 2022 22:56:05 +0000 (17:56 -0500)]
Update: Get the controller program in a better state for a stable release.

I started realizing that there were things that I forgot all about that the control and controller programs need to do.
There are too many changes to make immediately before the release.

Remove the incomplete code to avoid having stale code and therefore stale binary data.
Comment out the parts that generate socket related information as the socket cannot be supported yet.

2 years agoUpdate: Get the control program in a better state and update the control/controller...
Kevin Day [Sat, 2 Apr 2022 22:48:44 +0000 (17:48 -0500)]
Update: Get the control program in a better state and update the control/controller documentation.

I started realizing that there were things that I forgot all about that the control and controller programs need to do.
This is too many changes to make immediately before the release.

Wrap up the control program into a tolerable state and it will be published as is.
Release notes will be used to document that this program didn't make the cut in time.

2 years agoRegression: Invalid code added to controller.
Kevin Day [Sat, 2 Apr 2022 22:47:55 +0000 (17:47 -0500)]
Regression: Invalid code added to controller.

This got accidentally changed in this commit: 928bb81378ba50f086af5eb54327819751a523aa.

2 years agoBugfix: Incorrectly checking byte width when already inside a character.
Kevin Day [Sat, 2 Apr 2022 14:03:20 +0000 (09:03 -0500)]
Bugfix: Incorrectly checking byte width when already inside a character.

The width has already been determined, so these values are parts of a UTF-8 character.
The code is doing a byte check and attempting to verify or process this as a new UTF-8 character sequence.
This results in an inaccurate data from that point forward for the rest of the file.

2 years agoUpdate: Do not use sizeof(char_t) for explicitly handling 1 byte.
Kevin Day [Sat, 2 Apr 2022 13:41:07 +0000 (08:41 -0500)]
Update: Do not use sizeof(char_t) for explicitly handling 1 byte.

The char_t could potentially be of some size other than 1.
This read is per-byte and not per-character.
Normally these or true, but some special architectures this might not be.

2 years agoBugfix: The byte_dump program should respect the +q/++quiet.
Kevin Day [Sat, 2 Apr 2022 13:39:08 +0000 (08:39 -0500)]
Bugfix: The byte_dump program should respect the +q/++quiet.

When printing the "Invalid UTF-8 code.." or "read() failed.." error message, the code needs to check and respect the verbosity state.

2 years agoRegression: Accidentally reversed the big vs little endian bits.
Kevin Day [Sat, 2 Apr 2022 13:16:39 +0000 (08:16 -0500)]
Regression: Accidentally reversed the big vs little endian bits.

I had a fear that I might of done this with the commit dda60cc8d9cad69b55e7d79b4c052677b03b4413.
Turns out I was right.
All that back and forth getting confused before I realized that I should not use the BIG_ENDIAN macro did result in me getting confused on what bit goes where.

2 years agoUpdate: Centralize common error function, minor cleanups, and improve status code...
Kevin Day [Sat, 2 Apr 2022 04:08:41 +0000 (23:08 -0500)]
Update: Centralize common error function, minor cleanups, and improve status code programs.

Printing an error for two different long parameters that cannot be used together is very common.
Move this into a single function inside of fll_program.
Make sure that these calls are wrapped in appropriate verbosity checks.

The status code programs is now able to print the number with the error bit and/or warning bit set.
The status code programs status codes error and warning can be used together when using the -n/--number.

Perform minor cleanup where the "main->parameters.remaining.used > 0" can be simplified.

2 years agoRegression: Invalid syntax in fake program.
Kevin Day [Sat, 2 Apr 2022 03:53:45 +0000 (22:53 -0500)]
Regression: Invalid syntax in fake program.

I thought I compiled this to confirm that the code was correct.
Apparently not.

Fix the syntax so that the code compiles.

2 years agoUpdate: Add F_once, F_receive, F_repeat, and F_send status codes.
Kevin Day [Sat, 2 Apr 2022 02:52:38 +0000 (21:52 -0500)]
Update: Add F_once, F_receive, F_repeat, and F_send status codes.

This includes the *_not inverse versions.
Move the F_search into the correct position.

2 years agoUpdate: Remove inefficient and pointless size increase.
Kevin Day [Fri, 1 Apr 2022 04:48:29 +0000 (23:48 -0500)]
Update: Remove inefficient and pointless size increase.

This must have been by accident or by habit.
The (range->stop - range->start) for these shouldn't be happening.
These checks are solely for ensuring there is enough room after what is already allocated.
The behavior is actually potentially doubling its size.

Only increase size by the required amount.
There are likely other such problems that need to be fixed across this project.
I will likely not get to these by the 0.6.0 stable release and will have to address them following that as I find them.

2 years agoUpdate: Ensure all programs are handling signals, adding signal_received to fll_progr...
Kevin Day [Fri, 1 Apr 2022 04:42:01 +0000 (23:42 -0500)]
Update: Ensure all programs are handling signals, adding signal_received to fll_program_data_t.

Make sure fll_program_data_t supports storing the signal.
Move the commonly written functions that are pretty much identical across programs into fll_program:
- fll_program_standard_signal_received()
- fll_program_standard_signal_state()

Increase standard signal check from 10000 to 20000.
I really dislike the signal_check logic that exists purely as a work-around to the signal handling.
Most likely the only way to do this nicely that is available without rewritting the kernel and the libc will be to implement threads.
This will not be done for the 0.6.x versions and will likely happen for the 0.8.x versions.

2 years agoUpdate: Move the program data into the main for control.
Kevin Day [Thu, 31 Mar 2022 04:16:55 +0000 (23:16 -0500)]
Update: Move the program data into the main for control.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for control.
Kevin Day [Thu, 31 Mar 2022 03:45:34 +0000 (22:45 -0500)]
Update: Move the program data into the main for control.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Partially move program data into the main for controller.
Kevin Day [Thu, 31 Mar 2022 03:42:23 +0000 (22:42 -0500)]
Update: Partially move program data into the main for controller.

This does not use fll_program_data.
The controller needs a more complex structure than the one provided by fll_program_data.

The parts that can be moved are moved to be more consistent with other FLL provided programs.

2 years agoUpdate: Add pid integer to fll_program_data_t.
Kevin Day [Thu, 31 Mar 2022 03:28:43 +0000 (22:28 -0500)]
Update: Add pid integer to fll_program_data_t.

2 years agoUpdate: Move the program data into the main for fake.
Kevin Day [Thu, 31 Mar 2022 03:15:42 +0000 (22:15 -0500)]
Update: Move the program data into the main for fake.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Add umask integer to fll_program_data_t.
Kevin Day [Thu, 31 Mar 2022 01:31:52 +0000 (20:31 -0500)]
Update: Add umask integer to fll_program_data_t.

This should be a standard practice that ideally must go away if the POSIX standard ever fixes that idiotic umask() design.
Never under any circumstances should a read-only operation result require a write operation!
I am forced to work around this, so I might as well provide a standard practice work-around.

2 years agoUpdate: Move the program data into the main for firewall.
Kevin Day [Thu, 31 Mar 2022 01:21:38 +0000 (20:21 -0500)]
Update: Move the program data into the main for firewall.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Add child integer to fll_program_data_t.
Kevin Day [Thu, 31 Mar 2022 00:48:03 +0000 (19:48 -0500)]
Update: Add child integer to fll_program_data_t.

A few of the programs practice returning the child return status or the child process id.
Make this behavio standard by adding child to the fll_program_data_t.

2 years agoUpdate: Remove redundant F_status_set_error() when returning status with an error.
Kevin Day [Thu, 31 Mar 2022 00:20:06 +0000 (19:20 -0500)]
Update: Remove redundant F_status_set_error() when returning status with an error.

2 years agoUpdate: Use const for f_console_arguments_t and add envp.
Kevin Day [Thu, 31 Mar 2022 00:19:26 +0000 (19:19 -0500)]
Update: Use const for f_console_arguments_t and add envp.

2 years agoUpdate: Move the program data into the main for fss_basic_list_read.
Kevin Day [Thu, 31 Mar 2022 00:14:55 +0000 (19:14 -0500)]
Update: Move the program data into the main for fss_basic_list_read.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_basic_list_write.
Kevin Day [Thu, 31 Mar 2022 00:13:02 +0000 (19:13 -0500)]
Update: Move the program data into the main for fss_basic_list_write.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_basic_read.
Kevin Day [Thu, 31 Mar 2022 00:10:06 +0000 (19:10 -0500)]
Update: Move the program data into the main for fss_basic_read.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoCleanup: Fix styling, add new line before break.
Kevin Day [Thu, 31 Mar 2022 00:09:09 +0000 (19:09 -0500)]
Cleanup: Fix styling, add new line before break.

2 years agoSecurity: Invalid write in FSS processing functions due to improper allocation size.
Kevin Day [Thu, 31 Mar 2022 00:06:18 +0000 (19:06 -0500)]
Security: Invalid write in FSS processing functions due to improper allocation size.

The start and stop ranges are inclusive.
This means that the size is (stop - start) + 1.

The problems happens where the code is adding additional digits to represent end of line or other special characters.
When this is added, I seem to have forgotten to add the additional numbers to the + 1 and instead replaced the + 1.
This results in the size being potentially short by a single character and thus an invalid write is possible.

2 years agoUpdate: Move the program data into the main for fss_basic_write.
Kevin Day [Wed, 30 Mar 2022 23:53:15 +0000 (18:53 -0500)]
Update: Move the program data into the main for fss_basic_write.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_embedded_list_read.
Kevin Day [Wed, 30 Mar 2022 23:47:13 +0000 (18:47 -0500)]
Update: Move the program data into the main for fss_embedded_list_read.

Replace fss_embedded_list_read_main_t with fll_program_data_t and fss_embedded_list_read_data_t.
Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_embedded_list_write.
Kevin Day [Wed, 30 Mar 2022 04:58:31 +0000 (23:58 -0500)]
Update: Move the program data into the main for fss_embedded_list_write.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoCleanup: Add missing F_attribute_visibility_internal_d.
Kevin Day [Wed, 30 Mar 2022 04:57:32 +0000 (23:57 -0500)]
Cleanup: Add missing F_attribute_visibility_internal_d.

2 years agoUpdate: Move the program data into the main for fss_extended_list_read.
Kevin Day [Wed, 30 Mar 2022 04:12:55 +0000 (23:12 -0500)]
Update: Move the program data into the main for fss_extended_list_read.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoCleanup: Remove unnecessary error assignment.
Kevin Day [Wed, 30 Mar 2022 04:10:19 +0000 (23:10 -0500)]
Cleanup: Remove unnecessary error assignment.

2 years agoUpdate: Move the program data into the main for fss_extended_list_write.
Kevin Day [Wed, 30 Mar 2022 04:09:26 +0000 (23:09 -0500)]
Update: Move the program data into the main for fss_extended_list_write.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoCleanup: Remove unnecessary error assignment.
Kevin Day [Wed, 30 Mar 2022 04:08:54 +0000 (23:08 -0500)]
Cleanup: Remove unnecessary error assignment.

2 years agoUpdate: Move the program data into the main for fss_extended_read.
Kevin Day [Wed, 30 Mar 2022 04:04:52 +0000 (23:04 -0500)]
Update: Move the program data into the main for fss_extended_read.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_extended_write.
Kevin Day [Wed, 30 Mar 2022 04:02:31 +0000 (23:02 -0500)]
Update: Move the program data into the main for fss_extended_write.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_identify.
Kevin Day [Wed, 30 Mar 2022 04:00:19 +0000 (23:00 -0500)]
Update: Move the program data into the main for fss_identify.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_payload_read.
Kevin Day [Wed, 30 Mar 2022 03:55:04 +0000 (22:55 -0500)]
Update: Move the program data into the main for fss_payload_read.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main for fss_payload_write.
Kevin Day [Wed, 30 Mar 2022 03:54:34 +0000 (22:54 -0500)]
Update: Move the program data into the main for fss_payload_write.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoCleanup: Remove delete function notes from documentation comments.
Kevin Day [Wed, 30 Mar 2022 03:53:42 +0000 (22:53 -0500)]
Cleanup: Remove delete function notes from documentation comments.

2 years agoBugfix: The out of range is printing and then continues processing.
Kevin Day [Wed, 30 Mar 2022 03:37:07 +0000 (22:37 -0500)]
Bugfix: The out of range is printing and then continues processing.

When out of range happens an error should be printed and should stop processing for that parameter.
This is not happening because the status is incorrectly being returned when status is not assigned an error bit.
The out of range check happens independent of the status variable.

2 years agoUpdate: Move the program data into the main for fss_status_code.
Kevin Day [Wed, 30 Mar 2022 03:33:41 +0000 (22:33 -0500)]
Update: Move the program data into the main for fss_status_code.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Deallocate before releasing signals.
Kevin Day [Wed, 30 Mar 2022 03:30:01 +0000 (22:30 -0500)]
Update: Deallocate before releasing signals.

This helps ensure that signals, if being controlled, are properly deallocated before allowing any signals free range.

2 years agoUpdate: Restructure iki_read to use fll_program_data_t.
Kevin Day [Wed, 30 Mar 2022 03:17:30 +0000 (22:17 -0500)]
Update: Restructure iki_read to use fll_program_data_t.

Create iki_read_data_t for private data and passing the fll_program_data_t pointer.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Restructure iki_write to use fll_program_data_t.
Kevin Day [Wed, 30 Mar 2022 02:47:47 +0000 (21:47 -0500)]
Update: Restructure iki_write to use fll_program_data_t.

Create iki_write_data_t for private data and passing the fll_program_data_t pointer.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Move the program data into the main.
Kevin Day [Wed, 30 Mar 2022 02:42:07 +0000 (21:42 -0500)]
Update: Move the program data into the main.

Move parameter initialization and deallocation for FLL program data into main.c.

2 years agoUpdate: Make Utf8 private data private and move the program data into the main.
Kevin Day [Wed, 30 Mar 2022 02:10:44 +0000 (21:10 -0500)]
Update: Make Utf8 private data private and move the program data into the main.

Move parameter initialization and deallocation for fll program data into main.c.

Move the utf8 data into the private files.

2 years agoUpdate: Use signal_check more consistently in programs.
Kevin Day [Wed, 30 Mar 2022 01:17:09 +0000 (20:17 -0500)]
Update: Use signal_check more consistently in programs.

There are programs that do not use the fll_program_data_t structure.
These now define the signal_check in their main.
Such cases will go away as soon as they are converted into using fll_program_data_t.

2 years agoRegression: Use correct function for mutex lock.
Kevin Day [Wed, 30 Mar 2022 01:08:02 +0000 (20:08 -0500)]
Regression: Use correct function for mutex lock.

The lock is a mutex lock and not a lock.

2 years agoSecurity: Controller program is not initializing the alert mutex lock.
Kevin Day [Tue, 29 Mar 2022 23:23:25 +0000 (18:23 -0500)]
Security: Controller program is not initializing the alert mutex lock.

This results in undefined behavior and a possible segfault.

This appears to be an oversight.

Also remove stale commented out code and slightly improve the wording in a comment.

2 years agoUpdate: Finish writing tests, fix endianness, simplify some code.
Kevin Day [Tue, 29 Mar 2022 03:38:43 +0000 (22:38 -0500)]
Update: Finish writing tests, fix endianness, simplify some code.

Now that f_char_t exists and is in use, rename f_utf_character_t to f_utf_char_t to be consistent (and simpler).

The endianness detection is incorrect.
The BIG_ENDIAN is not represented as expected.
Change the detection to use GCC's __BYTE_ORDER__ macros.
Fallback to little endian if those macros are not available.

It took a while to figure out this was the problem and I got rather confused going back and forth between byte orders.
I think the changes I made are correct, but I need additional better testing on the byte orders.

2 years agoUpdate: Bring f_utf project in sync with f_string.
Kevin Day [Mon, 28 Mar 2022 04:30:30 +0000 (23:30 -0500)]
Update: Bring f_utf project in sync with f_string.

I have been explicitly avoiding making change to f_utf until I finished updating and successfully vetted the f_string project.
Now that the f_string has tests that pass, mass update the f_utf project.

This changes the functions and structure to be inline with how f_string works.
Many of the fixes are brought over.
I have observed that several of the bugs that were recently in f_string is also present in f_utf.
These bugs in f_utf should now be fixed as well.

This tests are brought over.
Only the tests for functions that are shared between f_string and f_utf are implemented.
And then those tests currently cannot be compiled.
The reason is that normal quoted strings cannot be used with the special UTF dynamic strings.
I simply did not have the time to get to this to finish writing these tests for f_utf.

There are some parts of f_string that make no sense in f_utf, such as f_string_range_t and f_string_quantity_t (both of which can be used on f_string and f_utf strings).
These have been omitted when copying the relevant f_string into f_utf code.

2 years agoCleanup: The f_string project.
Kevin Day [Mon, 28 Mar 2022 04:27:56 +0000 (23:27 -0500)]
Cleanup: The f_string project.

2 years agoUpdate: Restructure parts of f_utf project, and use f_utf_t, and add 'u' to hexdigits...
Kevin Day [Mon, 28 Mar 2022 00:56:35 +0000 (19:56 -0500)]
Update: Restructure parts of f_utf project, and use f_utf_t, and add 'u' to hexdigits in common.h.

Move the relevant functions into utf/convert.h, utf/is.h, and utf/is_character.h.

Implement f_utf_t as a type of uint32_t.
This should allow more customizability on that type if need be.
It also provides a more explicit context.

Add 'u' to designate the hexdigits are unsigned (This likely more specifically means it is an unsigned int).
This might be removed if it becomes a problem with architectures where "u" represents 16-bits or less.
Due to the size of the work, I am avoiding making this change in all of the Unicode comparison functions.
There are quite a lot of hexdigits in use.

2 years agoUpdate: Add remaining tests for f_string project.
Kevin Day [Sun, 27 Mar 2022 22:29:33 +0000 (17:29 -0500)]
Update: Add remaining tests for f_string project.

2 years agoBugfix: Partial string functions are incorrect.
Kevin Day [Sun, 27 Mar 2022 22:22:18 +0000 (17:22 -0500)]
Bugfix: Partial string functions are incorrect.

The tests exposed these problems.

The *_assure() functions are operating as if they are *_assure_nulless().
Remove the NULL checks that should not be there and fix the documentation comments.

The *_assure() functions are not considering the following:
1) The case where range.stop is >= source.used.
2) The fact that range.stop is inclusive and should not be directly used in the same way that *.used is used.

In the case of (1), if the stop range result in it overflowing past the actual length, the previous code results in invalid reads.
In the case of (2), the stop.range needs to instead be stop.range + 1.

2 years agoUpdate: Add additional tests for f_string project.
Kevin Day [Sun, 27 Mar 2022 16:00:28 +0000 (11:00 -0500)]
Update: Add additional tests for f_string project.

This adds tests for the f_string functions for functions defined in string.h.
This adds tests for the f_string functions for functions defined in dynamic.h that functionally mirror functions defined in string.h.

There are still more functions in dynamic.h that need to have tests written for them.

2 years agoBugfix: Incorrectly performing nulless string appends and prepends.
Kevin Day [Sun, 27 Mar 2022 15:19:04 +0000 (10:19 -0500)]
Bugfix: Incorrectly performing nulless string appends and prepends.

These problems are exposed by the tests that I am writing.

The destination->used < length check cannot be performed because of the presence of NULL characters.
The nulless versions of the strings may be identical even if their sizes do not match.
The only case where this check should still happen is when destination.used is 0.

The comparison operator is using the wrong variable in some cases ('j' should be used with 'destination').

Replace uint8_t with f_char_t.

Minor updates to the documentation comments.

2 years agoProgress: Controller and Control sockets.
Kevin Day [Sun, 27 Mar 2022 03:14:28 +0000 (22:14 -0500)]
Progress: Controller and Control sockets.

2 years agoBugfix: The f_socket_connect() needs to handle EINVAL errno.
Kevin Day [Sun, 27 Mar 2022 03:07:09 +0000 (22:07 -0500)]
Bugfix: The f_socket_connect() needs to handle EINVAL errno.

2 years agoBugfix: Print functions are not always printing the entire string.
Kevin Day [Sun, 27 Mar 2022 01:41:37 +0000 (20:41 -0500)]
Bugfix: Print functions are not always printing the entire string.

I observed this when working with printing binary data in '%Q'.

The fwrite_unlocked() might prematurely return but not on an error.
If it does, first check to see if it is an error and then keep trying until everything is printed.

This is more of a band-aid fix.
This requires an additional variable and an additional loop.
I would rather avoid all of this and so a re-design is likely necessary for greater efficiency.
As this is not my focus at the time I am going to tolerate this less than desirable design.

2 years agoBugfix: Increasing strings by too much.
Kevin Day [Sat, 26 Mar 2022 16:22:39 +0000 (11:22 -0500)]
Bugfix: Increasing strings by too much.

The *_increase_by() methods already include the ".used" length.
The previous code is also adding the ".used" length resulting in an increase of ".used" * 2.

2 years agoProgress: Controller and Control sockets.
Kevin Day [Sat, 26 Mar 2022 03:43:46 +0000 (22:43 -0500)]
Progress: Controller and Control sockets.

2 years agoCleanup: Clarify documentation comment that the function appends to the destination.
Kevin Day [Sat, 26 Mar 2022 03:42:11 +0000 (22:42 -0500)]
Cleanup: Clarify documentation comment that the function appends to the destination.

2 years agoBugfix: Invalid F_parameter error in f_socket functions.
Kevin Day [Sat, 26 Mar 2022 03:41:11 +0000 (22:41 -0500)]
Bugfix: Invalid F_parameter error in f_socket functions.

For these functions, the length pointer is optional and therefore must allow the length pointer to be 0.

2 years agoBugfix: Invalid '%lu', in formatted print, should instead be '%ul'.
Kevin Day [Sat, 26 Mar 2022 03:03:18 +0000 (22:03 -0500)]
Bugfix: Invalid '%lu', in formatted print, should instead be '%ul'.

2 years agoBugfix: When using --at with --total for --content, the returned number is always...
Kevin Day [Sat, 26 Mar 2022 02:38:08 +0000 (21:38 -0500)]
Bugfix: When using --at with --total for --content, the returned number is always 1 and should not be.

The code need to count all newlines rather than always returning 1.
This is likely a bug from copying the code from fss_basic_read where the non-zero count would indeed always be 1.

2 years agoBugfix: Invalid content count in FSS Basic List.
Kevin Day [Sat, 26 Mar 2022 02:34:34 +0000 (21:34 -0500)]
Bugfix: Invalid content count in FSS Basic List.

When F_none_eos or F_none_stop is returned, the content ends up having an additional size than the actual size.
This is happening because found->used++ is incremented immediately before returning.
When either F_none_eos or F_none_stop is returned, the content is not to be incremented.
Only the caller should know when to assume this is a proper ending representing the content end.

The macro private_macro_fl_fss_content_return_on_overflow_delimited() is only used in one place so remove it.

Make sure to initialize the content for all content read functions to ensure the content count is consistent.

2 years agoCleanup: Restructure to have the *_type_name() functions into public functions under...
Kevin Day [Fri, 18 Mar 2022 02:56:39 +0000 (21:56 -0500)]
Cleanup: Restructure to have the *_type_name() functions into public functions under common.h or similar.

Have the programs expose these functions to the public API.

The enumerations, consequently must also be exposed publicly.

2 years agoProgress: Controller and Control sockets.
Kevin Day [Fri, 18 Mar 2022 00:01:14 +0000 (19:01 -0500)]
Progress: Controller and Control sockets.

2 years agoProgress: Controller and Control sockets.
Kevin Day [Thu, 17 Mar 2022 03:56:05 +0000 (22:56 -0500)]
Progress: Controller and Control sockets.

2 years agoCleanup: Add a version number to the Open Standard License draft.
Kevin Day [Thu, 17 Mar 2022 03:55:16 +0000 (22:55 -0500)]
Cleanup: Add a version number to the Open Standard License draft.

2 years agoUpdate: Use return rather than assignment and break in type name functions.
Kevin Day [Thu, 17 Mar 2022 03:54:43 +0000 (22:54 -0500)]
Update: Use return rather than assignment and break in type name functions.

2 years agoProgress: Continue working on control and controller.
Kevin Day [Wed, 16 Mar 2022 04:34:00 +0000 (23:34 -0500)]
Progress: Continue working on control and controller.

This focuses on the control end of things.
This pulls in several dependencies that are needed to process the packet.

2 years agoUpdate: Packet specification.
Kevin Day [Tue, 15 Mar 2022 04:43:36 +0000 (23:43 -0500)]
Update: Packet specification.

2 years agoUpdate: Add F_status status type and rename F_status_size_max_with_signal to F_status...
Kevin Day [Tue, 15 Mar 2022 04:38:07 +0000 (23:38 -0500)]
Update: Add F_status status type and rename F_status_size_max_with_signal to F_status_size_max_with_bits_d.

The max size for what is now called F_status_size_max_with_bits_d is (2^16)-1 and not 2^16.

2 years agoUpdate: Add clang support to the bootstrap-example.sh script.
Kevin Day [Tue, 15 Mar 2022 04:35:33 +0000 (23:35 -0500)]
Update: Add clang support to the bootstrap-example.sh script.