| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
| |
no_ci
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
| |
no_ci
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
no_ci
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*out |= (n[i] & 0x7f) << (i * 7);
(n[i] & 0x7f) - will be an int value (32bit in most cases) and if
(i * 7) is larger than 32, we will shift outside of 32bit and 0
will be written into *out which is not what we want. To prevent
this, left operand of << should be first casted to 64bit and then
shift should be performed.
Marking no_ci since this code is not used in embedlog, and is
provided as an example for decoding numbers in binary logs.
no_ci
Reported-by: pvs-studio
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
Version should be changed in branch, and v9999 should indicate
master - unstable software. Good for daily builds.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
no_buildbot
|
|
|
|
|
|
|
|
|
| |
Although EL_FILE_SYNC_* has been renamed to EL_FSYNC_*, it is wise
to leave old enum for backward compatibility. It doesn't make code
any bigger.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
no_buildbot
|
|
|
|
|
|
|
|
| |
When no data has been written to file, there is not need to
perform fsync() syscall, which in this case will only eat
cycles for context switch without doing anything.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
|
| |
When funcinfo is being printed, 4 characters are printed beside
function name: [()]. But EL_PRE_FUNCINFO_LEN was configured to
add only 3 characters, which may lead to buffer overflow in corner
cases.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
EL_PRE_FUNCINFO_LEN was missing when calculating EL_PRE_LEN, which
is used to calculate EL_BUF_MAX. Without that, buffer was too small
to hold all print information, which could result in buffer overflow.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
|
| |
In case when message, finfo and colors are enabled and are full
(their text is as long as defined max values) it was possible to
overflow buffer. Very rare situation but surely it will happend,
to someone one day. Not anymore, it won't.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
| |
Cmon... it's pretty pointless and random at best. It's quite the
miracle it never failed before.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
embedlog.h is now generated, so include paths must be changed so
that path to generated file is passed. This is needed to make
out of tree compilation work.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
|
| |
This is readability patch, el_pmemory_table() was calling
el_opmemory_table() which was calling el_pmem(). Such proxy
is not needed and lower readability. Now el_pmemory_table()
calls el_pmem() directly with no proxy.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
|
|
| |
Previously parallel was creating directory based on options, but
we have so many options now, that length of directory name is larger
than 260 chars, and FS returns error. Now directory will be named
after seq number. Info about seq number and what options is associated
with it, will bo stored in compilation-test-results.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
stdout can take a lot of space, and if we multiply it by number
of compilation we do it will take TONS of memory just to hold
unneded data.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
This is a long test, and printing data to file took a lot of time.
Printing to buffer (so ram) shortens the test by order of magnitude.
It also tests custom print for binary data as a bonus.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
Previously binary data could be printed only to file, not it's possible
to print it to custom function as well. For that el_custom_put has been
extended with "size_t slen" to provide information about buffer size.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
|
| |
- it allows for compilation tests without pushing to remote
- clones are faster
- clone doesn't crash when tere are a lot of parallel clones (server
rejects connections)
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
compilation-test-results will be generated in the directory
where test-compilation.sh is, with information which flags
did succed and which did not.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
Without that -I../include, clang were loading system-wide embedlog.h
which could result in errror in case when system embedlog.h is
different than local one.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
| |
embedlog now can be safely used in multi-thread environment, that
implements pthread.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
print_mix_of_everything() takes a long time, and it looks like
it hung. This patch changes all mt_fok() checks to mt_run() so,
message for each check is printed.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
pbinary_mix_of_everything() takes a long time, and it looks like
it hung. This patch changes all mt_fok() checks to mt_run() so,
message for each check is printed.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
Previously default value was set (time()) which resulted in
fraction never being tested, since time() returns only
seconds, without fractions.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
| |
print_check() contained checks only for usec which resulted
in false check errors.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
| |
if (flags & 0x04 != 0x04) is invalid, as 0x04 != 0x04 will be
evaluated first.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
|
|
|
| |
When -ansi flag is enabled, configure won't find snprintf() function
and will set NEED_SNPRINTF_ONLY. But with it "credits" variable
will not be present in compilation unit as it requries
PREFER_PORTABLE_SNPRINTF - which we don't want to set. So
"str_arg = credits;" should be used only when PREFER_PORTABLE_SNPRINTF
is defined.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
|
|
|
| |
This patch removes check for functions that we cannot live
without anyway. Like - it's pointles to check for strlen(),
if that function is not available, program will not compile
no matter what. But checks like access(), if these functions
are not available, we can use different one (usually slower),
and we can still work.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
|
|
|
|
| |
All functions that are part of library public API, have now
comment stating that fact. It's easier now to know if function
can be called by external user or not.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|
|
|
|
| |
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
|