Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

require: honor "./path" #106

Closed
rurban opened this issue Jan 21, 2016 · 1 comment
Closed

require: honor "./path" #106

rurban opened this issue Jan 21, 2016 · 1 comment
Assignees
@rurban
Labels
bug security

Comments

@rurban
Copy link
Member

rurban commented Jan 21, 2016

don't walk INC with "./path" appended, try first the local path as if "." existed as first INC member.
same argumentation as in #58, but even more surprising. detected by jd and toddr

however it is documented that it does:

pp_ctl.c:
/* require doesn't search for absolute names, or when the name is
   explicitly relative the current directory */
PERL_STATIC_INLINE bool
S_path_is_searchable(const char *name)
@rurban rurban self-assigned this Jan 21, 2016
@rurban
Copy link
Member Author

rurban commented Feb 7, 2016

The report is bogus. ./ is honored properly already.

@rurban rurban closed this as completed Feb 7, 2016
rurban added a commit that referenced this issue Aug 27, 2018
@rurban
Cpanel-JSON-XS-4.06
1160402 
4.06 2018-08-22 (rurban)
        - Fix overloaded eq/ne comparisons (GH #116 by demerphq, GH #117 by Graham Knopp):
          detect strings, protect from endless recursion. false is now ne "True".
          clarify eq/ne rules in the docs.

4.05 2018-08-19 (rurban)
        - Set decoded type (PR #115 by Pali)
        - Add json_type_weaken (PR #114 by Pali)
        - Fix tests for 5.6 (rurban, pali)

4.04 2018-06-22 (rurban)
        - Fix bignum NaN/inf handling (#78 reported by Slaven Rezic)
        - Move author tests to xt/ as suggested in #106, added a make xtest target.
          Fixes a test fail with ASAN.

4.03 2018-06-21 (rurban)
        - Add sereal cpanel_json_xs type (#110 James Rouzier)
        - Fix bencode/bdecode methods in cpanel_json_xs (#111 Fulvio Scapin)
        - Overload ne operator for JSON::PP::Boolean (#107 tevfik1903)
        - Add a missing semicolon to a documentation example (#104 E. Choroba)

4.02 2018-02-27 (rurban)
        - Add encoder indent_length method (#103 rouzier), previously
          hard-coded to 3.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rurban rurban

Labels
bug security
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rurban