Ahmed Sozzer (Unofficial) / Ahmed Sozzer (Unofficial) is on Nostr

Unofficial automated RSS/Atom links to Ahmed Sozzer's work on Security, firmware, and maker projects. Original work is credited at the source; follow the source links. Zaps support the independent feed operator, not the author.

Last Notes

2026-10-22 02:43:35 GMT

Finding the PHP versions in your repos that need the php2shell fix
php.net shipped the fixes for php2shell (CVE-2026-109618 and CVE-2026-109619) on 21 Oct. To patch, you need a list of every PHP you run, and the versions written in your repos mostly can’t give you one. Across 10 public repos, 112 PHP version declarations were labeled before running the tool, and 111 of them can’t prove which PHP build actually runs.
So the work splits in two: find every place a repo declares PHP, then work out which ones you can fix by changing a line and which ones you have to check on a real build.
The fixed versions, and a 60-second check
From php.net’s advisory:
Branch
Fixed in
8.5
8.5.12
8.4
8.4.27
8.3
8.3.36
8.2
8.2.35
8.1 and older get no official release. php.net published source backport branches for 7.4, 8.0 and 8.1 (PHP-8.1-CVE-2026-109618-109619 in php-src, and the same pattern for the other two), and some distros and LTS vendors backport.
The advisory puts the highest risk on installs that serve web requests: PHP-FPM and mod_php first, then php-cgi, FrankenPHP and LiteSpeed. CI runners and dev machines still need the update. They’re just less exposed. If you can’t update yet, the advisory lists mitigations. php.net says they reduce exposure and don’t fix the bug, so I’m linking them instead of copying them here.
To see what a host or image actually runs:
# on a server or container
php -r 'echo PHP_VERSION, PHP_EOL;'
dpkg -l 'php*' 2>/dev/null | grep ^ii # Debian/Ubuntu packages
apk info -v 2>/dev/null | grep ^php # Alpine packages
# for a built image
docker run --rm --entrypoint php <image> -r 'echo PHP_VERSION, PHP_EOL;'
# images on a floating tag only pick up the fix after a rebuild and redeploy
docker build --pull .
For official images and source builds, PHP_VERSION is the answer. For distro packages it isn’t. Debian, Ubuntu and RHEL backport fixes without changing the upstream version, so compare the package version with your vendor’s security tracker. If you install from Remi’s repo, it had no updates as of 21 Oct.
Where PHP versions hide in a repo
That check needs a list of hosts and images to run it on, and the repo is where that list comes from. Each place PHP shows up says something different about the build.
Exact official tags. FROM php:8.3.30-fpm is as close to proof as a repo gets. The docker-library images are built from source at that version, so anything built from this line runs 8.3.30. Whether that’s the image you deployed is a separate question.
Floating tags. FROM php:8.3-fpm resolves at build time. You get the fix once the official image is rebuilt on 8.3.36 and you rebuild with --pull and redeploy. Restarting the same container keeps the old build, and so does reusing a tag with imagePullPolicy: IfNotPresent.
setup-php. php-version: '8.3.30' looks like a pin. setup-php installs whatever 8.3 patch is current or already on the runner, so a version at or above the fix can still run an older build. The first draft of the tool’s spec treated these as exact pins. The tool treats them as floating.
Version files. asdf installs exactly what .tool-versions says, so php 8.4.20 there is a real pin. .php-version is different: setup-php, Symfony CLI and Herd read it and use whatever patch is installed.
Distro packages. apt-get install php8.2-cli gets you whatever the vendor ships, and the version string won’t show a backported fix.
Images that bundle PHP. wordpress, drupal, nextcloud and Laravel Sail runtimes have PHP inside and no PHP version on the FROM line. The fix arrives when the maintainer rebuilds and you pull.
Composer constraints. "php": "^8.2" tells Composer what the code supports. It doesn’t pick the runtime. Updating Laravel, Symfony or WordPress code doesn’t change the runtime either, since php2shell is in PHP itself.
EOL branches. 8.1 and older need action however they’re declared: upgrade, take a vendor backport, or build from php.net’s backport branch.
What I measured
The corpus is 10 public PHP repos, each pinned to a commit SHA. The labels were made by reading each repo’s files line by line before running the tool on it, with an AI coding assistant, in one labeling pass. They’re recorded in results/labels.json, line level. Recall only covers the file types the labeling grep covered: Dockerfiles, Containerfiles, YAML, .php-version, .tool-versions and composer.json. The baseline is grep -rnE "FROM +php:", the usual “inventory your Dockerfiles” step. I’m not publishing per-repo verdicts.
Six repos were the tuning set. On the first run the tool found 58 of 69 labeled declarations, with 3 false positives, and grep found 31. The miss worth knowing about even if you never run the tool: setup-php versions that aren’t written next to setup-php. Some were set under env: and read back as ${{ env.php-version }}, and some were passed as an input to a reusable workflow (with: php-versions: "['8.3', '8.4']"). Search a workflow for setup-php and you’ll find the step, but the version lives somewhere else.
After fixing those gaps, the tuning set went to 70 of 70. That’s tuned on the same repos, so it doesn’t count. (It’s 70 now because the labeling pass missed one declaration, which was added after the first run.) Then 4 more repos were labeled the same way and measured once:
Held out: 4 repos, 42 labels, one run
Found
False positives
php2shell-repo-audit
36 / 42 (86%)
1
grep -rnE "FROM +php:"
16 / 42 (38%)
0
The 6 held-out misses were setup-php referenced by full URL (uses: https://github.com/shivammathur/setup-php@v2, common on Forgejo), an Arch pacman -S install, and Debian’s libapache2-mod-php. That last one is mod_php, one of the two highest-risk setups in the advisory, and its package name doesn’t start with php. All 6 are fixed in the release, so 36 of 42 measures the code from before those fixes.
Across all 10 repos, 111 of the 112 labeled declarations can’t prove which PHP build runs: floating tags, setup-php, distro packages, Sail runtimes, bundled images, templated tags and composer constraints. The one exception is a .tool-versions file with a patch version. Against php.net’s fix table, a naive version compare gives a firm “vulnerable” or “fixed” on all 95 version-dependent, non-EOL findings whose source can’t prove the runtime. Each of those is a guess.
It’s a small corpus with one labeling pass. Every run is in results/runs/, with the method in results/numbers.md.
The tool
php2shell-repo-audit reads those files and puts each declaration in ACTION REQUIRED, CHECK MANUALLY or DECLARED ≥ FIXED, with a reason. It never calls a repo fine: DECLARED ≥ FIXED always means “runtime not verified”. It needs Node 20 or newer and has no runtime dependencies.
npx [email protected] path/to/repo
npx [email protected] --org your-org --markdown > php2shell.md
--org reads default branches with a token from GH_TOKEN, GITHUB_TOKEN or gh auth token, GET requests only, and skips archived repos and forks. It only sees what the token can read, so check the repo count it prints.
Here’s the published 0.1.0 on the demo repo, which is synthetic: a made-up repo with 7 planted declarations.
$ npx [email protected] demo-repo
php2shell-repo-audit 0.1.0 · fix table rev 2: published 2026-10-21, https://www.php.net/archive/2026.php#2026-10-21-5 (remote)
Reads PHP versions declared in this repo. It does not check running servers.
ACTION REQUIRED (3)
.tool-versions:2 dev php 8.4.20 vulnerable: fixed in 8.4.27, upgrade
docker/api/Dockerfile:1 prod FROM php:8.3.30-fpm vulnerable: fixed in 8.3.36, upgrade
docker/legacy/Dockerfile:1 prod FROM php:8.1-fpm eol: no official php.net release for 8.1; upgrade, use your vendor's backport, or build from php.net's backport branch PHP-8.1-CVE-2026-109618-109619
CHECK MANUALLY (4)
.github/workflows/ci.yml:10 ci setup-php 8.3 floating-rebuild: setup-php installs the current or pre-installed 8.3 patch, not a pinned build
Dockerfile:1 prod FROM php:8.3-fpm floating-rebuild: rebuild with --pull and redeploy after the release, then confirm with the docker run check
composer.json:6 prod "php": "^8.2" constraint-only: a composer constraint is not the runtime
docker/worker/Dockerfile:3 prod apt-get install php8.2-cli distro-vendor: vendors backport fixes without changing the version; check your vendor's security tracker
DECLARED ≥ FIXED (0)
The docker run check: docker run --rm --entrypoint php <image> -r 'echo PHP_VERSION;'
exit 1: action required
grep finds 3 of those 7 lines. The two exact pins below the fix and the 8.1 image get a firm verdict. Everything the repo can’t settle goes to CHECK MANUALLY with the thing to check.
As a GitHub Action:
- uses: actions/checkout@v4
- uses: Amsozzer1/[email protected]
with:
path: .
fail-on: manual # or "action" to fail only on ACTION REQUIRED
It writes the findings to the job summary. By default it fails on ACTION REQUIRED, on anything that needs a manual check, and when it finds no PHP at all. A green job means nothing this repo declares needs attention, and says nothing about the runtime.
What it doesn’t see
It reads git. That leaves out:
Servers and shared hosting that aren’t described in a repo: cPanel, Plesk, .htaccess PHP handlers.
PaaS and managed runtimes such as Platform.sh, Upsun, Laravel Vapor, Bref, Forge and Heroku. The README has the full list.
Helm and Kustomize templates, compose extends:, GitLab include:, and versions set in .env files.
The image you actually deployed, and anything built outside the repo. That’s what the docker run line is for.
With --org, anything that isn’t on a default branch. A service built from another branch or an old tag won’t be in the list.
The next thing I’d build is an inspect <image> command that runs php inside an image and checks its distro packages, so CI can check the build instead of the file that describes it. Until that exists, the repo gives you the list of things to rebuild, and the docker run line is still the only step here that looks at a real build.
https://amsozzer.com/writing/finding-the-php-versions-that-need-the-php2shell-fix
#php #phpversions #fix #ubuntu #branchfixed
#rss #feeds #linksharing #qwibbit

2026-10-16 02:41:02 GMT

How often public Supabase repos leave a table readable by the anon key
While going through some of my older projects, I found a Supabase table with row level security off. It came in with a PR I’d mostly vibe coded, and I’d never turned RLS on for it. Anyone holding the anon key could read it, and the anon key ships in the frontend bundle. I fixed it as soon as I saw it.
Then I wanted to know how common that is. So I replayed the migration history of 1,000 public Supabase repos into throwaway local databases and checked every table, commit by commit.
Short version: 18.8% of the repos I could replay had a fully readable table at some point, and 10.9% still have one at their latest commit.
Why this happens
A Supabase frontend talks to Postgres through PostgREST, using the anon key. Supabase’s default privileges grant anon SELECT, INSERT, UPDATE and DELETE on new tables in public. Row level security is the only thing standing between those grants and your rows.
The dashboard’s table editor turns RLS on for new tables by default. A create table in a migration file doesn’t:
create table public.messages (
id bigint generated always as identity primary key,
body text not null
);
-- no "alter table public.messages enable row level security;"
-- anon can now read and write every row
So the same table is safe or open depending on whether you made it in the UI or in SQL.
What I measured
Sample. GitHub code search for .sql files under supabase/migrations/, sliced into 8 file-size bands to get past the API’s 1,000-result cap. Every band hit the cap, so the frame is the first 1,000 hits in each band: 5,865 repos. I screened them in seeded random order and kept the first 1,000 that weren’t forks, templates, tutorials or Supabase’s own examples, and that had at least 3 migration files and 2 commits touching them.
Replay. For each repo, every commit that touched the migrations or seed.sql, capped at 30 (first, last and evenly spaced in between). Each commit was replayed into a fresh supabase/postgres container with no outbound network, using the same rules supabase db push uses. Nothing touched a live project, and no URL was visited.
Exposure. A table in public with RLS off, on which anon has table-level SELECT. That means every row and every column is readable with the public key.
Results
Repos
Share
95% CI
Exposed at any sampled commit
146 of 777
18.8%
16.2 to 21.7%
Still exposed at the latest commit
43 of 393
10.9%
8.2 to 14.4%
The denominators differ because not every commit replays (more on that below). 777 repos had at least one commit that replays, and 393 had a latest commit that does.
When an exposure ended, it usually wasn’t a quick fix in the next commit. 67% of the ones that ended lasted more than a day and 37% more than a week. Counting exposures that are still open, the median was 47 days (Kaplan–Meier, since open ones haven’t finished yet). 22 of the 75 that ended did so at the very next migration commit.
How long exposures lasted, per repo, ended and still open
Ended exposures, 75 in all: under 1 hour 8, 1 hour to 1 day 17, 1 to 7 days 22, 7 to 30 days 9, 30 to 90 days 13, 90 to 365 days 5, over 1 year 1. Still open, time so far, 86 in all: under 1 hour 17, 1 hour to 1 day 11, 1 to 7 days 8, 7 to 30 days 12, 30 to 90 days 17, 90 to 365 days 20, over 1 year 1.
Under 1 hour
8 ended
17 still open
1 hour to 1 day
17
11
1 to 7 days
22
8
7 to 30 days
9
12
30 to 90 days
13
17
90 to 365 days
5
20
Over 1 year
1
1
How long exposures lasted, per repo. ‘Still open’ is the time so far at the last commit that replays. 161 exposures in 146 repos.
A few results worth a closer look
Starred repos did worse. Repos with 1 to 9 stars were exposed at their latest commit 20.3% of the time, against 8.3% for repos with no stars, and the intervals don’t overlap. I wouldn’t read that as cause. Starred repos tend to have longer histories, and every extra commit is another chance to add a table.
Older repos weren’t safer. Repos created before 2026 were at 16.7% at their latest commit and 2026 repos at 9.5%, and that difference is within the intervals. If you wanted to pin this on AI coding tools, this data doesn’t support it.
Most latest commits don’t replay at all. Only 39% of repos had a latest commit that applies cleanly on a fresh database. Of the ones that fail, 69% fail because a migration refers to a table, column or function that doesn’t exist yet, almost always because it was made in the dashboard. For those projects, the migration folder can’t rebuild the database it’s supposed to describe.
Public reads are everywhere, and that’s mostly fine. 48.6% of repos had a SELECT policy for anon with using (true) on a table that does have RLS on. That’s how a public profile page or a blog works. Supabase’s own linter skips that pattern on purpose and only flags always-true write policies, which showed up in 9.3% of repos.
Limits
Migrations aren’t the live database. A project can turn RLS on in the dashboard without a migration. Then it looks exposed here when it isn’t. The reverse happens too.
The sample is young and small. 72% of the repos were created in 2026 and 79% have no stars. Public repos lean toward hobby, demo and portfolio projects even after filtering.
The frame is whatever order GitHub search returns. Repos outside those 8 size bands, or past the first 1,000 hits in a band, had no chance of being picked.
The 30-commit cap can catch an exposure’s start late. I replayed 25 capped repos again with no cap (3,026 commits). No repo changed classification, and start times moved by at most about a day.
The 43
42 of the 43 repos still exposed at their latest commit got a private note, by email, GitHub’s private vulnerability reporting, or an issue asking for a security contact (with no details in it). The last one has no private contact channel and doesn’t look deployed, so I left it alone. The published data replaces repo names with repo-NNN and table names with salted hashes, because a distinctive table name can be searched back to its repo. As of October 15, one owner has confirmed a fix.
Checking your own project
In the dashboard, Advisors > Security flags these as “RLS disabled in public”. The fix is one line per table, plus policies for whatever access the app actually needs:
alter table public.messages enable row level security;
To catch it before it merges, I built Lintel, which replays a PR’s migrations and fails the check when the PR adds a table anon can read:
npx @amsozzer/lintel migrations check
There’s also a playground that runs the same checks in your browser.
The method, the scripts and the anonymized data (one row per sampled commit and per exposure) are in the repo: study/results.md.
https://amsozzer.com/writing/how-often-public-supabase-repos-leave-a-table-readable