summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* croco-config: Remove unused variableswip/fmuellner/coverage-fixesFlorian Müllner2019-02-191-5/+0
| | | | | | | | | "cflags" and "libs" were never used, "includedir" and "libdir" are unused since commit a17b0eb593. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* statement: Fix erroneous error checkFlorian Müllner2019-02-191-1/+2
| | | | | | | | | | The condition was already checked above. Instead, check that the last method call succeeded (but without leaking earlier allocated memory). Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* statement: Don't leak new statement objects on failureFlorian Müllner2019-02-191-0/+1
| | | | | | | | | cr_statement_new_with_media_rule() may fail and return NULL. The previously allocated memory should be freed in that case. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* statement: Fix string leaks in error caseFlorian Müllner2019-02-191-0/+6
| | | | | | Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* statement: Avoid leaks by early returnFlorian Müllner2019-02-191-6/+6
| | | | | | Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* om-parser: Fix string leak on errorFlorian Müllner2019-02-191-0/+2
| | | | | | | | | The string is taken by the declaration on success, but currently leaked on failure. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* om-parser: Don't leak charset in error caseFlorian Müllner2019-02-191-2/+2
| | | | | | | | | It is freed correctly if creating stmt2 fails, but leaked in case of stmt. Adjust the code so that the cleanup is run for both cases. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* om-parser: Fix double-free in error caseFlorian Müllner2019-02-191-3/+0
| | | | | | | | | Freeing decl is already handled under the error label, so don't free it before the jump. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* num: Don't leak duped num objects on failureFlorian Müllner2019-02-191-1/+2
| | | | | | | | | cr_num_dup() may fail and return NULL. The previously allocated memory should be freed in that case. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* parser: Don't leak tokenizers on failureFlorian Müllner2019-02-191-0/+8
| | | | | | | On success, the parser takes ownership of the tokenizer, but we still have to release it on failure. Spotted by covscan.
* parser: Don't leak new parser objects on failureFlorian Müllner2019-02-191-0/+2
| | | | | | | | | cr_parser_new() may fail and return NULL. The previously allocated memory should be freed in that case. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* parser: Don't leak error in error caseFlorian Müllner2019-02-191-2/+2
| | | | | | | | | | | RECORD_INITIAL_POS() returns in case of failure, and we leak the previously created error as a result. Fix by calling the macro before allocating the error. Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* parser: Don't leak pseudo in error caseFlorian Müllner2019-02-191-1/+3
| | | | | | Spotted by covscan. https://gitlab.gnome.org/GNOME/libcroco/merge_requests/1
* Visual Studio builds: Enhance security of x64 binariesChun-wei Fan2017-10-201-0/+4
| | | | | | | Use the HIGHENTROPYVA linker option on x64 builds with MSVC 2012 and later to enhance the security of the built binaries. Pointed out by Ignacio Casal Quinteiro.
* win32/replace.py: Fix replacing items in files with UTF-8 contentChun-wei Fan2017-05-091-2/+8
| | | | | | | | | Some files that this script will process might have UTF-8 items in there, which can cause problems on Python 3.x as it is more strict and careful on unicode issues. Fix this by: -Doing what we did before on Python 2.x -Opening the file with encoding='utf-8' on Python 3.x
* tknzr: support only max long rgb valuesIgnacio Casal Quinteiro2017-04-161-0/+10
| | | | | This fixes a possible out of bound when reading rgbs which are longer than the support MAXLONG
* tknzr: remove unneeded assignIgnacio Casal Quinteiro2017-04-161-1/+0
|
* input: check end of input before reading a byteIgnacio Casal Quinteiro2017-04-161-2/+9
| | | | | | When reading bytes we weren't check that the index wasn't out of bound and this could produce an invalid read which could deal to a security bug.
* Post release version bumpIgnacio Casal Quinteiro2017-04-061-1/+1
|
* Release 0.6.120.6.12Ignacio Casal Quinteiro2017-04-061-0/+4
|
* Typo fix s/unknow/unknown/Mattia Rizzolo2017-03-262-4/+4
| | | | | | Signed-off-by: Mattia Rizzolo <mattia@debian.org> https://bugzilla.gnome.org/show_bug.cgi?id=776798
* Typo fix s/Instanciation/instantiation/Mattia Rizzolo2017-03-262-7/+7
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=776798
* Typo fix s/occured/occurred/Mattia Rizzolo2017-03-265-7/+7
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=776798
* Visual Studio builds: Generate .pc filesChun-wei Fan2017-03-0312-13/+120
| | | | | | | | | Generate and copy the libcroco-0.6.pc during the "install" stage of the build, when a Python installation is found at the configured location in croco-version-paths.[vsprops|props]. Also fix the 201x "install" project as a "'" is missed, and clean up things a bit.
* Visual Studio builds: Add .pc generation scriptsChun-wei Fan2017-03-014-2/+269
| | | | | | | This adds Python scripts to generate .pc files for libcroco, so that it is easier for people who need to use the .pc files for the package. Integration to generate the .pc files will be added later to the project files.
* Visual Studio builds: Get the micro version as wellChun-wei Fan2017-03-012-2/+26
| | | | | | Update croco-version-paths.[vsprops|props].in to record the major, minor and micro versions of a release, and assemble the API version from these values. This prepares for generating the .pc file for libcroco.
* configure.ac: Bump LIBCROCO_MICRO_VERSIONChun-wei Fan2017-03-011-1/+1
| | | | We need to bump the micro version here as well...
* Visual Studio builds: Move projects to win32/Chun-wei Fan2017-02-2333-291/+290
| | | | | | This enables us to go one less layer down the tree to reach the projects and allows the autotools modules Makefile-newvs.am and Makefile.msvcproj be in sync with the latest ones in GLib master.
* Visual Studio builds: Support Visual Studio 2017Chun-wei Fan2017-02-155-6/+61
| | | | | | | | | | | | | This updates the autotools scripts so that we can support Visual Studio 2017 by copying the 2010 projects and update items in there as needed. Note that the format of the toolset version string changed for Visual Studio 2017, so allow a custom toolset version string to be passed in and used, otherwise the platform toolset string is generated as before. Note also that Visual Studio 2017 aims to be compatible with 2015 on the CRT level, so one should be able to use 2017-built binaries with 2015-built binaries without problems.
* Remove croco-install.props's during re-generationChun-wei Fan2016-10-191-0/+3
| | | | | | This is to ensure the changes in the generated build/win32/vs10/croco-install.props are applied in build/win32/vs[11|12|14] as well.
* build/: Force MSVC project file generation on Makefile.am changesChun-wei Fan2016-10-192-3/+10
| | | | | | | | | | | Make the Makefile.am targets for generating the Visual Studio projects re-generate the project files and the header listings whenever the Makefile.am's that include build/Makefile.msvcproj changes, so that whenever a source/header is added, they will be reflected in the projects and in the property sheets that are used to copy the headers. Also ensure that these are applied to the vs11, vs12 and vs14 projects when this happens, as they are copied and processed from the Visual Studio 2010 projects.
* Visual Studio builds: Rearrange include search pathsChun-wei Fan2016-06-232-2/+2
| | | | | Order the search sequence of the headers to conform better to the dependency hierarchy.
* Post release version bumpIgnacio Casal Quinteiro2015-12-171-1/+1
|
* Release 0.6.110.6.11Ignacio Casal Quinteiro2015-12-171-0/+4
|
* Fix regression in cr_tknzr_consume_chars()Florian Müllner2015-12-171-1/+1
| | | | | | | | | Commit bc9b2c339e introduced an intermediate variable to account for the difference in signedness between cr_tknzr_consume_chars() and cr_input_consume_chars(), but missed that the variable in question is an in-out variable that requires proper initialization. https://bugzilla.gnome.org/show_bug.cgi?id=759565
* Fix comparison warningMarek Chalupa2015-12-151-2/+2
| | | | | | Logical ! was only applied to the left side of the comparison https://bugzilla.gnome.org/show_bug.cgi?id=758394
* Post release version bumpIgnacio Casal Quinteiro2015-12-131-1/+1
|
* Release 0.6.100.6.10Ignacio Casal Quinteiro2015-12-131-0/+3
|
* Fix unused status variable in main()Руслан Ижбулатов2015-11-011-1/+1
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix unused status variable in evaluate_selectors()Руслан Ижбулатов2015-11-011-8/+32
| | | | | | | | | | * Add an end label to go to on error * Clean up xml_doc at the end * Fix a miscleaning of xpath_object * Unref sheets after they are given to a cascade * Return status at the end https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix unused status variable in set_prop_position_from_value()Руслан Ижбулатов2015-11-011-1/+1
| | | | | | * Return status https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix unused status variable in cr_utils_utf8_to_ucs1()Руслан Ижбулатов2015-11-011-3/+1
| | | | | | | * Don't set status to CR_OK, it's initialized to that value * Return status https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix unused status variable in cr_utils_ucs1_to_utf8()Руслан Ижбулатов2015-11-011-8/+2
| | | | | | | | * Return status instead (initialized to CR_OK) instead of an explicit CR_OK * Remove redundant check for *a_in_len < 1 (equivalent to *a_in_len == 0) and remove now-unused end label https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix cr_tknzr_consume_chars() to set *a_nb_char before returningРуслан Ижбулатов2015-11-011-2/+6
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix cr_input_consume_white_spaces to behave as documentedРуслан Ижбулатов2015-11-011-0/+2
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix improper use of g_ascii_strup()Руслан Ижбулатов2015-11-011-2/+1
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Derive pointer to have appropriate typeРуслан Ижбулатов2015-11-011-2/+1
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=710076
* Fix typosMarkus Engel2015-10-312-2/+2
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=726993
* Fix typo in logical conditionBoris Egorov2015-10-311-1/+1
| | | | | | | | | | | TYPE_SELECTOR equals to 2, so condition (cur_sel->type_mask | TYPE_SELECTOR) will always be true. We should increase c only if we encounter a selector. Issue detected by Cppcheck and MSVS (see https://bugzilla.gnome.org/show_bug.cgi?id=507484) Signed-off-by: Boris Egorov <egorov@linux.com>
* Post release version bumpIgnacio Casal Quinteiro2015-10-311-1/+1
|