Viewer triggers "You are not allowed to terraform parcel [PARCEL_NAME]" alert when terraforming on Edit Terrain: Everyone parcels

[maestrolinden]

Environment

Second Life Release 26.2.0.25386466510 (64bit)
Release Notes

You are at 196.5, 233.2, 23.0 in Rumpus Room 2 located at simhost-031b7244b46bd6761.agni
SLURL: https://maps.secondlife.com/secondlife/Rumpus%20Room%202/196/233/23
(global coordinates 331460.0, 306409.0, 23.0)
Second Life RC Magnum 2026-06-12.27437375581
Release Notes

CPU: Apple M1 Pro (2400 MHz)
Memory: 16384 MB
OS Version: macOS 26.5.1 Darwin 25.5.0 Darwin Kernel Version 25.5.0: Mon Apr 27 20:38:56 PDT 2026; root:xnu-12377.121.6~2/RELEASE_ARM64_T6000 arm64
Graphics Card Vendor: Apple
Graphics Card: Apple M1 Pro

OpenGL Version: 4.1 Metal - 90.5

Window size: 1065x707
Font Size Adjustment: 96pt
UI Scaling: 0.75
Draw distance: 72m
Bandwidth: 10000kbit/s
LOD factor: 1.125
Render quality: 1
Texture memory: 12124MB
Disk cache: Max size 2150.4 MB (24.3% used)
HiDPI display mode:

J2C Decoder Version: KDU v8.4.1
Audio Driver Version: OpenAL, version 1.1 ALSOFT 1.24.2 / OpenAL Community / OpenAL Soft: OpenAL Soft
Dullahan: 1.24.0.202510081737
CEF: 139.0.40+g465474a+chromium-139.0.7258.139
Chromium: 139.0.7258.139
LibVLC Version: 3.0.21
Voice Server Version: Secondlife WebRTC Gateway

Packets Lost: 0/29310 (0.0%)
June 22 2026 15:01:47

Description

When a parcel is set to allow anybody to terraform in its settings, the viewer repeatedly triggers an "You are not allowed to terraform parcel [PARCEL_NAME]" alert on each update, despite also sending a ModifyLand message for the operation and editing the terrain successfully.

The warning message is keyed to ParcelNoTerraforming internally, and is triggered locally by the viewer - not by the simulator.

This issue is a re-reporting of the 2018-era https://github.com/secondlife/jira-archive-internal/issues/60878 but with more details.

There's a 2nd (product) question about whether "Edit Terrain: Everyone" should be supported at all by the simulator, given that the SL Viewer dropped support for viewing and changing that parcel option in 2011: https://github.com/secondlife/jira-archive-internal/issues/131905 But at the very least, the viewer should be able to interact with simulators that do support the feature, such as the current release simulator build. I filed #5950 about this question.

Reproduction steps

Note: you can skip straight to step (5) if you visit https://maps.secondlife.com/secondlife/Rumpus%20Room%202/196/233/23 , which is currently set up for this bug repro.

1. UserA: Using a viewer that supports setting "Edit Terrain: Everyone" (such as Firestorm 7.2.4 (80712) - Second Life Viewers have not supported this option for 15 years), go to a parcel that you own, in a region that you do not own
   • Region ownership should be irrelevant but appears to be a factor - see comment about Claude's analysis below.
2. UserA: Assuming Firestorm's UI, open "World" -> "Parcel Details" -> "Options", and enable "Edit Terrain: Everyone"
3. UserA: Open "World" -> "Region Details" -> "Region", and ensure that "Block Terraform" is disabled
4. UserB: Visit UserA's parcel - any viewer build should work, including the latest Second Life Release 26.2.0.25386466510 or Firestorm 7.2.4 (80712) Release viewer
5. UserB: Open the land tab of the build tool, and select 'Raise'
6. UserB: Click and drag in UserA's parcel to raise the land, and note the results
   Expected results:
   Since "Edit Terrain: Everyone" is enabled and the feature is still (apparently) supported server-side, UserB should be able to edit the terrain without any warnings.

Actual results:

• UserA and UserB both see UserA's terrain edits take effect.
• UserA sees an erroneous stream of "You are not allowed to terraform parcel [PARCEL_NAME]" alerts for the parcel
• Using Hippolyzer, I can see that the viewer sends ModifyLand messages to edit the terrain, and that it also received a ParcelProperties message which includes the ALLOW_TERRAFORM flag among the options - which I believe corresponds to "Edit Terrain: Everyone":

ParcelProperties of test parcel from simulator

``` IN ParcelProperties [EQ]

[AgeVerificationBlock]
RegionDenyAgeUnverified = False
[MediaData]
MediaDesc = ''
MediaHeight = 0
MediaLoop = 1
MediaType = 'none/none'
MediaWidth = 0
ObscureMedia = 0
ObscureMusic = 0
[ParcelData]
AABBMax = <208.0, 236.0, 50.0>
AABBMin = <192.0, 224.0, 0.0>
AnyAVSounds = True
Area = 192
AuctionID = 0
AuthBuyerID = 00000000-0000-0000-0000-000000000000
Bitmap =| [b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x0f\x00',
b'\x00\x00\x00\x00\x00\x00\x0f\x00',
b'\x00\x00\x00\x00\x00\x00\x0f\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00',
b'\x00\x00\x00\x00\x00\x00\x00\x00']
#Bitmap = b'\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x0f\x00\x00\x00\x00\x00\x00\x00\x0f\x00\x00\x00\x00\x00\x00\x00\x0f\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00'
Category =| 'NONE' #0
ClaimDate =| '2026-06-22T13:38:14' #1782160694
ClaimPrice = 0
Desc = ''
GroupAVSounds = True
GroupID = 00000000-0000-0000-0000-000000000000
GroupPrims = 0
IsGroupOwned = False
LandingType =| 'NONE' #1
LocalID = 2
MaxPrims = 65
MediaAutoScale = 0
MediaID = 00000000-0000-0000-0000-000000000000
MediaURL = ''
MusicURL = ''
Name = 'terrain test'
OtherCleanTime = 0
OtherCount = 12
OtherPrims = 0
OwnerID = 39b44919-512a-40a5-8347-90f9125937e4
OwnerPrims = 0
ParcelFlags =| ('ALLOW_FLY',
'ALLOW_OTHER_SCRIPTS',
'ALLOW_LANDMARK',
'ALLOW_TERRAFORM',
'CREATE_OBJECTS',
'ALLOW_GROUP_SCRIPTS',
'CREATE_GROUP_OBJECTS',
'ALLOW_ALL_OBJECT_ENTRY',
'ALLOW_GROUP_OBJECT_ENTRY',
'ALLOW_VOICE_CHAT',
'USE_ESTATE_VOICE_CHAN') #0x7e00005b
ParcelPrimBonus = 1.0
PassHours = 1.0
PassPrice = 10
PublicCount = 0
RegionDenyAnonymous = False
RegionDenyIdentified = False
RegionDenyTransacted = False
RegionPushOverride = False
RentPrice = 0
RequestResult = 0
SalePrice = 0
SeeAVs = True
SelectedPrims = 0
SelfCount = 0
SequenceID = -10000
SimWideMaxPrims = 65
SimWideTotalPrims = 0
SnapSelection = True
SnapshotID = 00000000-0000-0000-0000-000000000000
Status =| 'LEASED' #0
TotalPrims = 0
UserLocation = <0.0, 0.0, 0.0>
UserLookAt = <0.0, 0.0, 0.0>
[ParcelEnvironmentBlock]
ParcelEnvironmentVersion = -1
RegionAllowEnvironmentOverride = True
[ParcelExtendedFlags]
Flags = b'\x00\x00\x00\x00'
[RegionAllowAccessBlock]
RegionAllowAccessOverride = True

</details>

[canny]

This issue has been linked to a Canny post: Terraforming permission error 🎉

[maestrolinden]

I probed Claude about this issue, and it offered the following opinion. Note that point (2) about parcel vs region ownership actually checks out; I originally couldn't reproduce this when testing in a region in which the parcel owner was also the region owner, but could reproduce it in a location with distinct parcel and region owners.

Yes, this is a clear viewer bug. Here's the precise breakdown:

The bug in canTerraformParcel()

bool LLToolBrushLand::canTerraformParcel(LLViewerRegion* regionp) const
{
    LLParcel* selected_parcel = LLViewerParcelMgr::getInstance()->getHoverParcel();
    bool is_terraform_allowed = false;
    if (selected_parcel)
    {
        bool owner_release = LLViewerParcelMgr::isParcelOwnedByAgent(selected_parcel, GP_LAND_ALLOW_EDIT_LAND);
        is_terraform_allowed = ( gAgent.canManageEstate() || (selected_parcel->getOwnerID() == regionp->getOwner()) || owner_release);
    }

    return is_terraform_allowed;
}

The three conditions that allow terraforming are:

1. gAgent.canManageEstate() — agent is estate manager ✓ sensible
2. selected_parcel->getOwnerID() == regionp->getOwner() — parcel owner equals region owner ✗ wrong
3. owner_release — isParcelOwnedByAgent(selected_parcel, GP_LAND_ALLOW_EDIT_LAND) ✓ sensible

Condition 2 is the bug. It compares the parcel's OwnerID against regionp->getOwner() (the estate/region owner's UUID), not against gAgent.getID(). So it only grants terraform if the parcel owner happens to be the same person who owns the region — which is irrelevant to whether the current agent owns the parcel.

Your message confirms this: OwnerID = 39b44919-... and ParcelFlags includes ALLOW_TERRAFORM. If the agent is the parcel owner but not the region owner, condition 2 is false, isParcelOwnedByAgent handles the agent-as-owner case via parcelp->getOwnerID() == gAgent.getID() in isParcelOwnedByAgent... wait — that actually should return true via owner_release. Let's check further.

Looking at isParcelOwnedByAgent:

bool LLViewerParcelMgr::isParcelOwnedByAgent(const LLParcel* parcelp, U64 group_proxy_power)
{
    if (!parcelp)       return false;
    if (gAgent.isGodlike())    return true;
    if (parcelp->getOwnerID() == gAgent.getID())    return true;
    if (parcelp->isPublic())   return false;
    return gAgent.hasPowerInGroup(parcelp->getOwnerID(), group_proxy_power);
}

This would return true if the agent is the parcel owner. So owner_release should catch the normal case. The real bug is then that condition 2 is simply wrong — it's checking if the parcel owner is the region owner, which is a meaningless check that:

• passes for the wrong people (non-agent region owners),
• is redundant with owner_release in the one case where it might make sense.

The actual ALLOW_TERRAFORM parcel flag is never consulted at all in canTerraformParcel(). The function checks ownership/estate status but completely ignores PF_ALLOW_TERRAFORM / getAllowTerraform(). This means:

• A parcel with ALLOW_TERRAFORM set (open to all) will still be blocked for non-owners
• There's no path in this function that reads the flag you see in the message

The correct fix would be to also check selected_parcel->getAllowTerraform() as a permitted condition:

c++
is_terraform_allowed = ( gAgent.canManageEstate() 
                       || selected_parcel->getAllowTerraform()   // missing!
                       || owner_release );

And condition 2 (getOwnerID() == regionp->getOwner()) should either be removed or corrected to getOwnerID() == gAgent.getID(), though owner_release already covers that correctly.