Release 2015.1

When encoding message, accept 2015.

Verify return code from rnet_encode.

If a given file header is not currently supported or has other problems
that we did not detect during first parsing, the user should be warned
and the program bail out.

Release memory when returning error during rnet_encode.

Do a beta release for 2015 support.

Support header for 2015.

As usual, fields have been added to the file header. Parsing has been
tested, but not transmitting.

Release v.2014.2.

Fixing copyright headers

I made a mistake in the last commit and used my work e-mail address in
the Copyright header of some files.  Fixing that.

Create manpages for the project

This commit creates the manpages for the project.  There are two
versions implemented here: the en_US, and the pt_BR.  Along with the
manpages, I am also modifying and creating Makefile.am files needed to
properly handle/install the manpages.

It is worth making some notes here:

- We have to keep an eye in order to sync the manpages every time one of
  them is modified.

- We also have to be careful to update the manpages every time a new
  option is added to the program.

Fix bug when saving the receipt file using a .DEC filename as template

This commit fixes a bug that happens when the user provides a .DEC
filename to be transmitted, and rnetclient uses this filename to compose
the receipt (.REC) filename to be saved.  I forgot to use "basename" to
extract only the .DEC filename (and strip all the other paths) before using
it to build the .REC full path.  As an addition, this commit also fixes
a small memory leak.  Tested by sending a declaration again.

Add .dir-locals.el

This comment adds the file .dir-locals.el, which is a special file
loaded by Emacs every time a file is visited.  For now, it just sets the
coding style to "linux".

Update/improve the way we check for libraries

Using PKG_CHECK_MODULES is not very neat because the user might want to
be able to provide a different $LDFLAGS, and this would break the build.
Instead, we should be using autoconf's default
AC_CHECK_LIB/AC_SEARCH_LIBS (in our case, I chosed the latter).  This
way, we don't need to modify $LIBS inside configure.ac.

This patch also adds a check for zlib, and updates $CFLAGS accordingly
based on the output of 'pkg-config' for both GNUTLS and zlib libraries.

Include extra compiler flags

Given the last attempts to clean up the code, this commit adds extra
compiler flags that are useful to keep the codebase sane.  The added
flags, for now, are:

  -Wall -Werror -Wextra -Wunused -Wunused-variable

Initialize variables (and silence GCC's -Wunused-variable warning)

Using -Wunused-variable to compile the code, GCC gives a false positive
about two variables in the code.  This commit initializes those
variables to sane values, in order to silence this warning.

Add comments to configure.ac

I think it is important to make configure.ac nicer to read, because
sometimes one has to hack it and decrypt the code in there.  This commit
achieves this goal by (a) adding the GPL license header, (b) commenting
the important steps of configure.ac, and (c) updating some parameters
from certain macros.

Update call to AM_INIT_AUTOMAKE

According to:

  <https://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_005fINIT_005fAUTOMAKE-invocation>

the call to AM_INIT_AUTOMAKE should be updated to take no arguments (in
our case), because the other forms will be deprecated.  This patch does
that.

Tested with an older version of automake to make sure nothing breaks.

Fix other warnings.

When using -Wextra, there were some signedness related warnings, as well
as size warnings, like using %li for pointer diff on 32-bit systems.

This was build-tested on both a 32-bit ARM and 64-bit x86_64.

Fix lots of compiler warnings (casts, unused vars, etc.)

As a preparation for a patch to create a spec file for the project (and
be able to generate RPM's for Fedora-like systems), this commit fixes
lots of warnings generated by the compiler when using more strict
flags.

Fedora, by default, uses the following flags to compile the program:

  -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
  -fstack-protector-strong --param=ssp-buffer-size=4
  -grecord-gcc-switches -m%ARCH -mtune=generic

This generated lots of warnings (like missing casts, or unused
variables).  It is good to fix those in order to keep a sane codebase.

Add GPL header to bootstrap script

When I added the bootstrap script, I forgot to add the GPL header to it.
I believe it is a good practice to do that for every file in the
project, so I am fixing it for this specific file.

Implement new '--output-dir' option.

This commit implements the new '--output-dir' (or '-o') option.  It can
be used to specify where the receipt of the submission will be saved.

Currently, rnetclient saves the receipt in your $HOME, with a generic
name which is composed using the CPF and some random digits.  This is
not good because the proprietary IRPF program expects the receipt name
to be the same as the declaration name, but with the extension renamed
from ".DEC" to ".REC".

This patch implements some new concepts.  First, if the user provides an
output directory in the command line, we save the receipt there.  If she
does not provide anything, then we save the receipt in the current
working dir (CWD), which is a more sensitive decision IMO.  Also, and
perhaps more important, is the fact that now the program automagically
detects when the filename has the ".DEC" extension, and uses the same
filename for the receipt in this case (replacing ".DEC" by ".REC", as
expected).  This makes the proprietary crap recognize our receipt
out-of-the-box, without having to worry with renamings.  It is worth
mentioning that if the user provides a declaration file which does not
have the ".DEC" extension, then rnetclient fallbacks to the old behavior
and saves the filename as "$CPF.REC" (I chose not to use random digits
in the end of the filename).

On a side note, I would like to say that I am not entirely happy with
the way we handle missing directories and files, but that is a topic for
a completely different patch...

Fix bug in argument parsing.

The argument '-d' is marked as having an optional argument to it, which
is wrong, because it is mandatory to provide the filename when you use
'-d'.

Release v2014.1.

Update copyright headers

Implement argument parsing via argp.h.

This commit implements argument parsing from the command line via
argp.h. So far, I have added only one parameter ('-d', or
'--declaracao') which is used for providing the filename of the
declaration.  I have also marked this argument as optional, so that the
user will still be able to provide the filename without using '-d'.

As an auxiliary support, I enabled the generation of the config.h header
file, so that the program can obtain the package version and the bug
report address directly from configure.ac.

Change bug report address to softwares-impostos@fsfla.org.

We have been using softwares-impostos@fsfla.org for patches and other
discussions about rnetclient for some time. It's better that people send
any messages about rnetclient to that mailing list rather than sending
them to only me.

Adding .gitignore

This adds .gitignore and makes it ignore build system files
(automatically generated) and object files generated by the compilation.

Add Alexandre and Sergio to list of AUTHORS.

Check return code for write when sending messages to server.

This not only avoids a warning, but prevents us from moving forward,
when we should report an error, when write fails.

Fix compiler warnings on %i format for long int.

On 64-bit platforms, we get the following warning:

decfile.c:213:3: warning: format ‘%i’ expects argument of type ‘int’, but argument 3 has type ‘long int’ [-Wformat]

Using %li works on both 32-bit ARM and 64-bit x86.

Add bootstrap.sh script

This commit adds the bootstrap.sh script, which automates the processing
of configure.ac and Makefile.am files in order to generate the build
scripts.

Support large file uploads

If the declaration file is larger than ~64KiB, upload it in multiple
separately-deflated chunks.

unsign char in message parsing

Cast to unsigned any chars read from the buffer while computing lengths
during rnet message parsing.

Remove unused variable while at that.

Check len macros during parsing.

Move header start and end macros to decfile.h as head and tail.
Add sanity checking code during parsing.  Adjust the encoder.
Update the format docs.

Use exerc instead of ano to choose encoding lengths

We key sizes on exerc, not on the calendar year the declaration refers
to, so compare exerc instead of ano.  Also, test latest years first.

Document operating system as GNU.

The code sets so to GNU, but the docs suggested Linux.  Fix the latter
to match.

Update cliente version to that of ReceitaNet 2014.

Odds are the reception side will reject IRPF2014 declarations transferred
by a ReceitaNet implementation that claims to be last year's.

Set versao_java to a fake gij string.

We're not running Java at all, so we might as well pretend to be the
GNU Interpreter for Java.

Send right part of the header for 2014.

The code was sending a piece of the header as expected by the server,
considering the header size for 2013. Now, we check for the year we are
submitting for, and use the proper size.

Reported-by: Alexandre Oliva <lxoliva@fsfla.org>

Use macros for the header size.

For each year, use a macro for the header size we expect.

Grow message by just the needed amount

Do not allocate more space than required for the message to grow.
We used to nearly double the amount of space every time, which is
nice to avoid quadratic complexity, but files are small enough
that it probably doesn't matter much, and it just makes things
confusing.

Convert chars to len with unsigned buffer

Introduce chars2len to compute a len out of an unsigned char buf[2].
This avoids problems when any of the chars is negative, if char is
signed.

Silence compiler warning in 64-bit mode

Casting an int to gnutls_transport_ptr_t gets a warning when
sizeof(void*)!=sizeof(int).  Casting the int to int_ptr first
silences the warning.

Add documentation about where to find header documentation.

The DEC file "documentation" can be found as a resource file at Receita
Federal IRPF JAR.

Release v2014.0.

Use the year in the release version, indicating support for that year.

Distinguish between 2013 and 2014.

The file format changes throughout the years, with added fields in the
header. Verify the size of the header and the year indicated, and add
support for year 2014.

Use errno to indicate error in parsing file.

Release v0.1.1.

Indicate a retifying field in the header.

Use the field in the DEC file to determine if this is a retifying file
or note, and use this in the hash in the first message.

Sixth byte in some messages are not always 1.

The first message, with the hash, uses the sixth byte as 1. The second
message, on the other hand, uses it as 0.

Handle uncompressed messages.

Some messages may come uncompressed, and we need to handle them.

Provide function to remove a header from a message.

Remove the first n bytes from a message.

Fix use of deflate output length.

avail_out has the remaining bytes in the buffer, not the number of bytes
output, which is found on total_out.

Update TODO.

Just to be safe, show message and try to save file.

When we receive a response we are not sure how to handle, try to find a
message and a file in the response and use them.

Handle error response.

Handle response when DEC has already been transmitted.

The response given by the server when the file has already been
transmitted includes a message to the user and the receipt file, which
is saved at the user's home or a temporary directory with a unique
filename.

Parse key/value message, extracting the values of its keys.

Check for responses and send second message.

This completes the protocol. The responses need to be decoded and
handled now.

Create helpers to send and receive messages.

Needs to do some input and error checking and maybe add some timeouts.

Fix inflate.

Inflate support was miscalculating the size of the output buffer.

Update TODO.

Add the missing field in the first message.

It's composed of part of the header register. It's more simple to build
it as a substring of the register itself, than composing it field by
field.

Fix size of field.

tam_dados_val_chave is, in fact, only 4 bytes long. Due to a known bug
in the decoder used during reverse engineering, the value of
tam_dados_val was appended to the value of tam_dados_val_chave, since
the two keys have the same prefix.

Fix key for type of identification.

It's called tp_ni, not tipo_ni.

Calculate message hash and use it.

Use gcrypt to calculate stripped file MD5 hash and add it to the
key/value message.

Allow an arbitrarily-sized buffer into the key/value message.

Add transmission length to the key/value message.

Use the length of the stripped file that is going to be transmitted for
some of the keys in the first message.

Strip the DEC file from control numbers.

In order to send the second part of the message, we need to strip the
declaration registers from their control numbers.

Fix bug when expanding message.

If we need to expand a message while adding a field to it, we need to
update the pointer we are using, and the one we are returning back.

Use a pointer for message functions when it needs a resize.

Always use a pointer to a pointer to struct rnet_message in those
functions where it is possible that it may be resized.

Add more fields to the first message.

The only missing fields are the size and hash of the second message and
part of the header register.

Add 64-bit values support for rnet_message.

Include some extra fields in the message.

These fields are mostly machine/software/network identity numbers. Some
choices here may be political. Perhaps we should let the user pick them
up at some point, as long as there are sensible defaults.

Allow a 16-bit value to be added to the key/value pair.

Send the 8-bit value address instead of its value.

Add more fields to the first message.

Allow a single byte value to be added as a key/pair value.

Encode some fields in the first protocol message.

Read some fields from the decfile and add those to a message built for
the ReceitaNet protocol.

Build rnet_message.

Fix exercicio_pgd in protocol documentation.

Due to a bug in the parser used for reverse engineering, the field named
"exercicio" was appended to the field "exercicio_pgd".

Create rnet_message to encode messages.

rnet_message allows us to encode messages in the form used for
ReceitaNet, which is a kind of set of key/values.

Parse DEC file header.

Include fields from the header in a pmhash and introduce a function to
retrieve them. Some of those fields are needed to send the first message
to the server.

Introduce poor man's hash.

This lookup array is helpful for adding fields parsed from the DEC file.

Remove handshake success message

Start parsing a DEC file.

Read lines from a DEC file and store them in an array for later parsing.

Update TODO

Update copyright years

Document server answer and next client message.

Complete format documentation.

Document message header.

Start of documentation about the protocol.

Create a function for the first handshake.

Use getaddrinfo to obtain receitanet's addresses.

The name address for receitanet is receitanet.receita.fazenda.gov.br.
Instead of using its IPv4 address directly, tries to resolve the name
and use the given addresses.

As a bonus, allows us to use /etc/hosts to debug with a test server.

Increase buffer size for testing.

For the current tests, the buffer needs to be increased, so the data
file can be read.

Use the right IP address for receitanet server

Verify connect return code.

If connect is not successful, exit.

First take at a client for ReceitaNet.