Skip to content

Invalid integer constant expression error in php.h #2466

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2,111 commits into from

Conversation

ointeractive-depot
Copy link

@ointeractive-depot ointeractive-depot commented Apr 10, 2017

Getting a error

C:\PHP\sdk\phpdev\vc14\x86\php-7.1.3-src\main/php.h(326): fatal error C1017: invalid integer constant expression.

when compile project at VC++ 2015

Change

# if PHP_WIN32

to

# ifndef PHP_WIN32

at 326 line in php.h will solve the problem.

commercebyte and others added 30 commits February 7, 2017 09:19
…buggy behavior,

when PHP returns "Undefined static property" error due to class entry corruption.
With my fix for bug 74053, both tests return no errors now, I corrected the EXPECTF accordingly

[Anybody please advice if I'm wrong?]

Also created bug74053.phpt, for the code I mentioned in the bug description
…object_store_no_reuse,

to avoid alignment issues
EG_FLAGS_IN_SHUTDOWN - is set when PHP is in shutdown state
* PHP-7.1:
  Update NEWs
  Fixed bug #74019 (Segfault with list)
…or spawns another object). (jim at commercebyte dot com)

Merge branch 'master' of github.com:commercebyte/php-src

* 'master' of github.com:commercebyte/php-src:
  Added EG(flags) - executor global flags EG_FLAGS_IN_SHUTDOWN - is set when PHP is in shutdown state
  newly added zend_object_store.no_reuse is redefined as a global zend_object_store_no_reuse, to avoid alignment issues
  Alignment fix, as per @nikic
  The test scripts bug64720.phpt and bug68652.phpt were relying on the buggy behavior, when PHP returns "Undefined static property" error due to class entry corruption. With my fix for bug 74053, both tests return no errors now, I corrected the EXPECTF accordingly
  Bug Fix: Corrupted class entries on shutdown when a destructor spawns another object (C) 2017 CommerceByte Consulting
* PHP-7.1:
  Fixed typo
* PHP-7.1:
  add missing datatype as sync for 5113909
This reverts commit d94c2c7.

WSA functions are documented explicitly as unsafe for dllmain
- avoid locking
- initialize only once
- the process will fail, if no init failed
This reverts commit 23bd7bc.

Looks like this change is unstable. If same CSP is use but multiple processers,
the initialization failures are possible. Thus, CryptAcquireContext in
every process, even if it won't be used at all, is not sensible. This
might actually motivate to look for better CSP APIs.
* PHP-7.1:
  Update NEWS
  Fixed #73496 (Invalid memory access in zend_inline_hash_func)
* PHP-7.1:
  Update NEWS
  Fixed bug #74084 (Out of bound read - zend_mm_alloc_small)
As in previous variant, locking is removed and the initialization
is done only once at process start. The CNG API turns out to be
faster, also the initialization is less resources hungry. The
initialization part could need to be improved, if too much startup
failures are sighted in the real world usage. Though that would mean
having locking back.

The usage of CNG was already pointed out and requested in several
reports, with the further refactoring it appears to make sense and
simplify things a backward compatible way.
The limit is big enough, and it's questionable such amount of
random data can ever make sense anyway.
Rework for 6095070, so then any encoding is supported. The path
segment length is measured in wchar_t size, whereby the number
of wchar_t is 255+\0. This means, in the actual encoding, the path
segment size can become (255*<bytes per glyph>)+\0 bytes in worst
case. It is still valid, as all the FS API uses wide chars
internally.
weltling and others added 22 commits April 2, 2017 16:40
fix makefile generation for non clang

rename option
* PHP-7.1:
  fix buffer overflow
* PHP-7.1:
  fix possible out of bounds buffer access
* PHP-7.1:
  Revert "extend skip section"
* PHP-7.1:
  Update NEWS
  Remove unused var
  Fix invalid free of persistent results on error/connection loss in mysqlnd (bug 74376)
* PHP-7.1:
  Fixed bug #74353 (Segfault when killing within bash script trap code)
That omits querying of short names, thus improving perf.
    increase test coverage in calendar extension
    consistent warnings in calendar extension
* PHP-7.1:
  Update NEWS
  Fixed bug #74379 (syntax error compile error in libmagic/apprentice.c)
  Missed NEWS

Conflicts:
	ext/fileinfo/libmagic.patch
@laruence laruence closed this Apr 10, 2017
php-pulls pushed a commit that referenced this pull request Apr 10, 2017
* PHP-7.0:
  Fixed issue #2466 Invalid integer constant expression error in php.h
php-pulls pushed a commit that referenced this pull request Apr 10, 2017
* PHP-7.1:
  Fixed issue #2466 Invalid integer constant expression error in php.h
php-pulls pushed a commit that referenced this pull request Apr 10, 2017
* 'master' of git.php.net:/php-src:
  Fixed condition check
  Fixed condition check
  another place
  Fixed issue #2466 Invalid integer constant expression error in php.h
php-pulls pushed a commit that referenced this pull request Apr 10, 2017
* 'PHP-7.0' of git.php.net:/php-src:
  Fixed condition check
  Fixed issue #2466 Invalid integer constant expression error in php.h
php-pulls pushed a commit that referenced this pull request Apr 10, 2017
* 'PHP-7.1' of git.php.net:/php-src:
  Fixed condition check
  Fixed condition check
  another place
  Fixed issue #2466 Invalid integer constant expression error in php.h
dstogov added a commit to zendtech/php-src that referenced this pull request Apr 10, 2017
* master:
  Remove spurious `CG(context).in_finally` dingleberry
  Forgot to commit test file...
  Implement jumptable optimization
  Support more than two successors in opcache CFG
  Fix JMPZ+JMP => NOP+JMP optimization
  Fixed bug #74404 (wrong reflection on DateTimeZone::getTransitions)
  Update NEWS
  Fixed bug #74402 (segfault on random_bytes, bin2hex, openssl_seal)
  intergarte furher newer flag in FindFirstFileEx for win7+ only
  Fixed bug #74400 (phpdbg comparing socket function to int)
  Fixed condition check
  Fixed condition check
  another place
  Fixed issue php#2466 Invalid integer constant expression error in php.h
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.